openstack
/
puppet-designate
Code Issues Proposed changes

designate-worker: Fix outdated topic parameter 

The [service:worker] worker_topic parameter was renamed to the topic
parameter during Train cycle[1]. This change updates the parameter
configured accordingly.

[1] fcb3290be7caf04b6b065f8a25bc4b8a3fd9113a

Closes-Bug: #1968523
Change-Id: I2da91700288682e35d78e2eea1983a55c73fc483
This commit is contained in:
Takashi Kajinami 2022-04-11 14:53:37 +09:00
parent c90e2d4e5d
commit 933517abcd
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manifests/worker.pp
View File

@ -118,7 +118,12 @@ class designate::worker(
'service:worker/poll_max_retries': value => $poll_max_retries;
'service:worker/poll_delay': value => $poll_delay;
'service:worker/export_synchronous': value => $export_synchronous;
'service:worker/worker_topic': value => $worker_topic;
'service:worker/topic': value => $worker_topic;
}
# TODO(tkajinam): Remove this after Zed release
designate_config {
'service:worker/worker_topic': ensure => absent;
}
if $worker_notify != undef {

4
spec/classes/designate_worker_spec.rb
View File

@ -54,7 +54,7 @@ describe 'designate::worker' do
is_expected.to contain_designate_config('service:worker/poll_max_retries').with_value( '<SERVICE DEFAULT>' )
is_expected.to contain_designate_config('service:worker/poll_delay').with_value( '<SERVICE DEFAULT>' )
is_expected.to contain_designate_config('service:worker/export_synchronous').with_value( '<SERVICE DEFAULT>' )
is_expected.to contain_designate_config('service:worker/worker_topic').with_value( '<SERVICE DEFAULT>' )
is_expected.to contain_designate_config('service:worker/topic').with_value( '<SERVICE DEFAULT>' )
end
end
@ -87,7 +87,7 @@ describe 'designate::worker' do
is_expected.to contain_designate_config('service:worker/poll_delay').with_value( params[:poll_delay] )
is_expected.to contain_designate_config('service:worker/notify').with_value( params[:worker_notify] )
is_expected.to contain_designate_config('service:worker/export_synchronous').with_value( params[:export_synchronous] )
is_expected.to contain_designate_config('service:worker/worker_topic').with_value( params[:worker_topic] )
is_expected.to contain_designate_config('service:worker/topic').with_value( params[:worker_topic] )
end
end