From: Mathieu Gagné <mgagne@iweb.com>
Date: Tue, 28 Jan 2014 23:18:00 +0000 (-0500)
Subject: Do not enforce glance_control_exchange
X-Git-Tag: 3.1.1~4
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=fa5a6ee45f973bc73e8a7e662b4b31a41f81ce5a;p=puppet-modules%2Fpuppet-ceilometer.git

Do not enforce glance_control_exchange

The default glance_control_exchange in Ceilometer has been changed
from 'glance_notifications' to 'glance' in grizzly-2. Glance always
had 'glance' for its control_exchange.

This was a bug in Ceilometer and puppet-ceilometer patched it.
It shouldn't anymore.

Change-Id: I217515c91f092fb3e8039dde2d14a7ab4cbb7245
Closes-bug: #1273878
(cherry picked from commit 85d46894cca6ba80e8cb57243dde57e44cffa443)
---

diff --git a/manifests/init.pp b/manifests/init.pp
index 70d4926..10e2100 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -180,9 +180,6 @@ class ceilometer(
     'DEFAULT/metering_secret'        : value => $metering_secret;
     'DEFAULT/debug'                  : value => $debug;
     'DEFAULT/verbose'                : value => $verbose;
-    # Fix a bad default value in ceilometer.
-    # Fixed in https://review.openstack.org/#/c/18487/
-    'DEFAULT/glance_control_exchange': value => 'glance';
     'DEFAULT/notification_topics'    : value => 'notifications';
   }
 
diff --git a/spec/classes/ceilometer_init_spec.rb b/spec/classes/ceilometer_init_spec.rb
index 9a19d51..977a5a1 100644
--- a/spec/classes/ceilometer_init_spec.rb
+++ b/spec/classes/ceilometer_init_spec.rb
@@ -153,10 +153,6 @@ describe 'ceilometer' do
       it { should contain_ceilometer_config('DEFAULT/syslog_log_facility').with_value('LOG_LOCAL0') }
     end
 
-    it 'fixes a bad value in ceilometer (glance_control_exchange)' do
-      should contain_ceilometer_config('DEFAULT/glance_control_exchange').with_value('glance')
-    end
-
     it 'configures notification_topics' do
       should contain_ceilometer_config('DEFAULT/notification_topics').with_value('notifications')
     end