openstack/puppet-aodh
Code Issues Proposed changes

Stop hard-coding config file for db sync

Browse Source 
The aodh-dbsync command by default loads /etc/aodh/aodh.conf. Removing
the override allows us to use additional paths such as aodh.conf.d in
the future.

Change-Id: Ic3bf121611d7221209c006872ab98b0682aa2273
This commit is contained in:
Takashi Kajinami 2024-03-05 12:18:57 +09:00
parent 9d14c5f8e1
commit b3016e3bf3
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 aodh::db::sync (
include aodh::deps
exec { 'aodh-db-sync':
command => 'aodh-dbsync --config-file /etc/aodh/aodh.conf',
command => ['aodh-dbsync'],
path => '/usr/bin',
refreshonly => true,
user => $user,

4
spec/classes/aodh_db_sync_spec.rb
View File

@ -8,7 +8,7 @@ describe 'aodh::db::sync' do
it 'runs aodh-db-sync' do
is_expected.to contain_exec('aodh-db-sync').with(
:command => 'aodh-dbsync --config-file /etc/aodh/aodh.conf',
:command => ['aodh-dbsync'],
:path => '/usr/bin',
:refreshonly => 'true',
:user => 'aodh',
@ -33,7 +33,7 @@ describe 'aodh::db::sync' do
it {
is_expected.to contain_exec('aodh-db-sync').with(
:command => 'aodh-dbsync --config-file /etc/aodh/aodh.conf',
:command => ['aodh-dbsync'],
:path => '/usr/bin',
:refreshonly => 'true',
:user => 'aodh',