Additional puppet-lint formatting

Change-Id: I6e5fa77a301eec30cff8e16bad33a91bfd95b13f
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
Reviewed-on: https://review.openstack.org/17176
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins

Rakefile

@@ -3,3 +3,6 @@ require 'puppetlabs_spec_helper/rake_tasks'
 require 'puppet-lint/tasks/puppet-lint'
 PuppetLint.configuration.fail_on_warnings = true
 PuppetLint.configuration.send('disable_80chars')
+
+# Disable check due to upstream bug: https://github.com/rodjek/puppet-lint/issues/170
+PuppetLint.configuration.send('disable_class_parameter_defaults')

modules/etherpad_lite/manifests/init.pp

@@ -46,7 +46,7 @@ class etherpad_lite (
     revision => $nodejs_version,
     require  => [
         Package['git'],
-        File["${base_install_dir}"],
+        File[$base_install_dir],
     ],
   }
 

modules/gerrit/manifests/init.pp

@@ -65,7 +65,7 @@
 #     Set this to true to disable cron jobs and replication,
 #     which can interfere with testing.
 # TODO: make more gerrit options configurable here
-
+#
 class gerrit(
   $war = '',
   $mysql_password = '',

modules/openstack_project/manifests/gerrit.pp

@@ -66,8 +66,8 @@ class openstack_project::gerrit (
     vhost_name                      => $vhost_name,
     canonicalweburl                 => $canonicalweburl,
     # opinions
-    enable_melody                   => 'true',
-    melody_session                  => 'true',
+    enable_melody                   => true,
+    melody_session                  => true,
     # passthrough
     ssl_cert_file                   => $ssl_cert_file,
     ssl_key_file                    => $ssl_key_file,

modules/pip/manifests/init.pp

@@ -1,12 +1,12 @@
+# Class: pip
+#
 class pip {
-
-  package { "python-all-dev":
-    ensure => present
-  }
-
-  package { "python-pip":
+  package { 'python-all-dev':
     ensure => present,
-    require => Package[python-all-dev]
   }
 
+  package { 'python-pip':
+    ensure  => present,
+    require => Package['python-all-dev'],
+  }
 }