Skip to content

Commit

Permalink
Merge pull request #945 from fatpat/fix_purge_parameter
Browse files Browse the repository at this point in the history
fix purge rabbitmq_parameter
  • Loading branch information
wyardley committed Oct 30, 2023
2 parents ba728f5 + 9995f42 commit d03c21a
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/puppet/provider/rabbitmq_parameter/rabbitmqctl.rb
Expand Up @@ -69,7 +69,7 @@ def set_parameter
key = resource[:name].rpartition('@').first

if @property_flush[:ensure] == :absent
rabbitmqctl('clear_parameter', '-p', vhost, resource[:component_name], key)
rabbitmqctl('clear_parameter', '-p', vhost, resource[:component_name] || component_name, key)
else
rabbitmqctl('set_parameter', '-p', vhost, resource[:component_name], key, resource[:value].to_json)
end
Expand Down
19 changes: 19 additions & 0 deletions spec/acceptance/parameter_spec.rb
Expand Up @@ -45,4 +45,23 @@ class { 'rabbitmq':
end
end
end

context 'destroy parameter resource' do
it 'runs successfully' do
pp = <<-EOS
rabbitmq_parameter { 'documentumFed@fedhost':
ensure => absent,
}
EOS

apply_manifest(pp, catch_failures: true)
apply_manifest(pp, catch_changes: true)
end

it 'does not have the parameter' do
shell('rabbitmqctl list_parameters -q') do |r|
expect(r.stdout).not_to match(%r{documentumFed\s+})
end
end
end
end

0 comments on commit d03c21a

Please sign in to comment.