Skip to content

Commit

Permalink
Do not enforce glance_control_exchange
Browse files Browse the repository at this point in the history
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 85d4689)
  • Loading branch information
mgagne committed Feb 28, 2014
1 parent 9510508 commit fa5a6ee
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
3 changes: 0 additions & 3 deletions manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -180,9 +180,6 @@
'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';
}

Expand Down
4 changes: 0 additions & 4 deletions spec/classes/ceilometer_init_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -153,10 +153,6 @@
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
Expand Down

0 comments on commit fa5a6ee

Please sign in to comment.