]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Define $user referenced in db.pp
authorYanis Guenane <yanis.guenane@enovance.com>
Thu, 20 Feb 2014 14:55:13 +0000 (09:55 -0500)
committerYanis Guenane <yanis.guenane@enovance.com>
Fri, 21 Feb 2014 19:26:06 +0000 (14:26 -0500)
In db.pp, in the exec 'ceilometer dbsync' ressource the user
is supposed to be $::ceilometer::params::user, but it was
not defined.

Change-Id: I812b6ba2877faddfce86d3c97b95772b0d22d368

manifests/db.pp
manifests/params.pp
spec/classes/ceilometer_db_spec.rb

index 1cf955aea0033b105aaba9b86bb78989f0f9f8c0..15822d8ce95f218ee805b3e7ed4df3f023506ef9 100644 (file)
@@ -62,7 +62,7 @@ class ceilometer::db (
   exec { 'ceilometer-dbsync':
     command     => $command,
     path        => '/usr/bin',
-    user        => $::ceilometer::params::username,
+    user        => $::ceilometer::params::user,
     refreshonly => true,
     logoutput   => on_failure,
     subscribe   => Ceilometer_config['database/connection']
index c240bb9da90ef613ce660755d4f3130214c576e5..89281be3248f89aa10a7227780319f18fe3cf8a2 100644 (file)
@@ -4,6 +4,7 @@ class ceilometer::params {
 
   $dbsync_command  = 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf'
   $expirer_command = 'ceilometer-expirer'
+  $user            = 'ceilometer'
 
   case $::osfamily {
     'RedHat': {
index aa3610010f91447c05049ce5201d118bb525d15b..0dc88dcd330f9ec2f650f62b667c5c81a3d32a6e 100644 (file)
@@ -27,6 +27,7 @@ describe 'ceilometer::db' do
         :command     => 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -60,6 +61,7 @@ describe 'ceilometer::db' do
         :command     => '/bin/true',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -92,6 +94,7 @@ describe 'ceilometer::db' do
         :command     => 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -125,6 +128,7 @@ describe 'ceilometer::db' do
         :command     => '/bin/true',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -154,6 +158,7 @@ describe 'ceilometer::db' do
         :command     => 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end