Define $user referenced in db.pp

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
xbezdick · Feb 21, 2014 · e6c1911 · e6c1911
1 parent 1aa1d79
commit e6c1911
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/manifests/db.pp b/manifests/db.pp
@@ -62,7 +62,7 @@
   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']

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -4,6 +4,7 @@
 
   $dbsync_command  = 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf'
   $expirer_command = 'ceilometer-expirer'
+  $user            = 'ceilometer'
 
   case $::osfamily {
     'RedHat': {

diff --git a/spec/classes/ceilometer_db_spec.rb b/spec/classes/ceilometer_db_spec.rb
@@ -27,6 +27,7 @@
         :command     => 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -60,6 +61,7 @@
         :command     => '/bin/true',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -92,6 +94,7 @@
         :command     => 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -125,6 +128,7 @@
         :command     => '/bin/true',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end
@@ -154,6 +158,7 @@
         :command     => 'ceilometer-dbsync --config-file=/etc/ceilometer/ceilometer.conf',
         :path        => '/usr/bin',
         :refreshonly => 'true',
+        :user        => 'ceilometer',
         :logoutput   => 'on_failure'
       )
     end