From 015fd676e45095d38ff3a0b0050aab96bafe7c94 Mon Sep 17 00:00:00 2001 From: ZhongShengping Date: Tue, 28 Mar 2017 16:21:30 +0800 Subject: [PATCH] Password should be secured As messaging_urls option contains rabbit_password value, it should be secured. Change-Id: I503180280a5558d27c00368e9949aaed52151e79 Closes-Bug: #1676708 --- manifests/agent/notification.pp | 2 +- spec/classes/ceilometer_agent_notification_spec.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/manifests/agent/notification.pp b/manifests/agent/notification.pp index 50421e8..6e95526 100644 --- a/manifests/agent/notification.pp +++ b/manifests/agent/notification.pp @@ -124,6 +124,6 @@ class ceilometer::agent::notification ( 'notification/store_events' : value => $store_events; 'notification/disable_non_metric_meters': value => $disable_non_metric_meters; 'notification/workers' : value => $notification_workers; - 'notification/messaging_urls' : value => $messaging_urls; + 'notification/messaging_urls' : value => $messaging_urls, secret => true; } } diff --git a/spec/classes/ceilometer_agent_notification_spec.rb b/spec/classes/ceilometer_agent_notification_spec.rb index d5800a6..0c0d905 100644 --- a/spec/classes/ceilometer_agent_notification_spec.rb +++ b/spec/classes/ceilometer_agent_notification_spec.rb @@ -109,7 +109,7 @@ describe 'ceilometer::agent::notification' do it 'configures two messaging urls' do is_expected.to contain_ceilometer_config('notification/messaging_urls').with_value( ['rabbit://rabbit_user:password@localhost/nova', 'rabbit://rabbit_user:password@localhost/neutron'] - ) + ).with_secret(true) end end -- 2.45.2