stackforge/puppet-openstack-cloud
Code Issues Proposed changes

volume/common: re-enable telemetry config

Browse Source 
It was disabled due to LP #1266259.

Close #112

Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>
This commit is contained in:
Emilien Macchi 2014-01-29 15:18:41 +01:00
parent 884ce27642
commit 6feaee109f
2 changed files with 2 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
manifests/volume.pp
View File

@ -44,12 +44,6 @@ class cloud::volume(
use_syslog => true
}
#TODO(EmilienM) While https://review.openstack.org/#/c/65008/ got merged
# we can't declare ceilometer class due to bug LP #1266259
# class { 'cinder::ceilometer': }
# Temp fix:
cinder_config {
'DEFAULT/notification_driver': value => 'cinder.openstack.common.notifier.rpc_notifier';
}
class { 'cinder::ceilometer': }
}

4
spec/classes/cloud_volume_controller_spec.rb
View File

@ -57,9 +57,7 @@ describe 'cloud::volume::controller' do
:log_facility => 'LOG_LOCAL0',
:use_syslog => true
)
should contain_cinder_config('DEFAULT/notification_driver').with('value' => 'cinder.openstack.common.notifier.rpc_notifier')
should contain_class('cinder::ceilometer')
end
it 'configure cinder scheduler' do