openstack/puppet-cinder
Code Issues Proposed changes

Fix path for cinder-manage

Browse Source 
When we run the db sync, grep is called in the process however it is not
found in the path as we are only setting /usr/bin

Change-Id: I9905e02958f715be1cfebaf73e0e93b65b267ef9
This commit is contained in:
Alex Schultz 2019-03-18 07:28:45 -06:00
parent 37d351359e
commit ce6682d5a9
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/db/sync.pp
View File

@ -17,7 +17,7 @@ class cinder::db::sync(
exec { 'cinder-manage db_sync': exec { 'cinder-manage db_sync':
command => "cinder-manage ${extra_params} db sync", command => "cinder-manage ${extra_params} db sync",
path => '/usr/bin', path => ['/bin', '/usr/bin'],
user => 'cinder', user => 'cinder',
refreshonly => true, refreshonly => true,
try_sleep => 5, try_sleep => 5,

4
spec/classes/cinder_db_sync_spec.rb
View File

@ -6,7 +6,7 @@ describe 'cinder::db::sync' do
is_expected.to contain_exec('cinder-manage db_sync').with( is_expected.to contain_exec('cinder-manage db_sync').with(
:command => 'cinder-manage db sync', :command => 'cinder-manage db sync',
:user => 'cinder', :user => 'cinder',
:path => '/usr/bin', :path => ['/bin', '/usr/bin'],
:refreshonly => 'true', :refreshonly => 'true',
:try_sleep => 5, :try_sleep => 5,
:tries => 10, :tries => 10,
@ -30,7 +30,7 @@ describe 'cinder::db::sync' do
is_expected.to contain_exec('cinder-manage db_sync').with( is_expected.to contain_exec('cinder-manage db_sync').with(
:command => 'cinder-manage --config-file /etc/cinder/cinder.conf db sync', :command => 'cinder-manage --config-file /etc/cinder/cinder.conf db sync',
:user => 'cinder', :user => 'cinder',
:path => '/usr/bin', :path => ['/bin', '/usr/bin'],
:refreshonly => 'true', :refreshonly => 'true',
:try_sleep => 5, :try_sleep => 5,
:tries => 10, :tries => 10,