From 850bc1b6859840c8a6a4b8b6bd5a36a2e69e5b11 Mon Sep 17 00:00:00 2001 From: Matthew Wagoner Date: Mon, 26 Nov 2012 11:13:11 -0500 Subject: [PATCH] Clean up of minor puppet-lint warnings. Mostly documentation and parameterised class parameter complaints. Change-Id: I5b3ffa4ad3f707f385165c2d86c891c6bd4c1ae0 Reviewed-on: https://review.openstack.org/16901 Reviewed-by: James E. Blair Approved: Clark Boylan Reviewed-by: Clark Boylan Tested-by: Jenkins --- manifests/conf.pp | 7 ++++--- manifests/init.pp | 7 ++++--- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/manifests/conf.pp b/manifests/conf.pp index 96ef42f..5e10422 100644 --- a/manifests/conf.pp +++ b/manifests/conf.pp @@ -1,3 +1,5 @@ +# == Define: ulimit::conf +# define ulimit::conf ( $limit_domain, $limit_item, @@ -10,8 +12,7 @@ define ulimit::conf ( content => template('ulimit/limits.erb'), replace => true, owner => 'root', - mode => 0644, - require => File['/etc/security/limits.d'] + mode => '0644', + require => File['/etc/security/limits.d'], } - } diff --git a/manifests/init.pp b/manifests/init.pp index 54ae9cb..edb5f13 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -1,13 +1,14 @@ +# == Class: ulimit +# class ulimit { package { ['libpam-modules', 'libpam-modules-bin']: - ensure => present + ensure => present, } file { '/etc/security/limits.d': ensure => directory, owner => 'root', - mode => 0755 + mode => '0755', } - }