openstack
/
puppet-ceilometer
Code Issues Proposed changes

Add 'ceilometer-service' tag to collector and agent/notification 

Set the 'ceilometer-service' tag for the collector and the
notification services as it happens already for the others.

Change-Id: I4c6fe871a1d55ea4dfdfc33edafdcf57bbd91e5e
This commit is contained in:
Giulio Fidente 2016-03-29 15:05:59 +02:00
parent 8142a6a019
commit 497aad5423
4 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
manifests/agent/notification.pp
View File

@ -90,7 +90,8 @@ class ceilometer::agent::notification (
name => $::ceilometer::params::agent_notification_service_name,
enable => $enabled,
hasstatus => true,
hasrestart => true
hasrestart => true,
tag => 'ceilometer-service'
}
ceilometer_config {

3
manifests/collector.pp
View File

@ -87,6 +87,7 @@ class ceilometer::collector (
name => $::ceilometer::params::collector_service_name,
enable => $enabled,
hasstatus => true,
hasrestart => true
hasrestart => true,
tag => 'ceilometer-service'
}
}

6
spec/classes/ceilometer_agent_notification_spec.rb
View File

@ -72,7 +72,8 @@ describe 'ceilometer::agent::notification' do
:name => platform_params[:agent_notification_service_name],
:enable => params[:enabled],
:hasstatus => true,
:hasrestart => true
:hasrestart => true,
:tag => 'ceilometer-service'
)
end
end
@ -91,7 +92,8 @@ describe 'ceilometer::agent::notification' do
:name => platform_params[:agent_notification_service_name],
:enable => false,
:hasstatus => true,
:hasrestart => true
:hasrestart => true,
:tag => 'ceilometer-service'
)
end
end

9
spec/classes/ceilometer_collector_spec.rb
View File

@ -68,7 +68,8 @@ describe 'ceilometer::collector' do
:name => platform_params[:collector_service_name],
:enable => true,
:hasstatus => true,
:hasrestart => true
:hasrestart => true,
:tag => 'ceilometer-service'
)
end
@ -87,7 +88,8 @@ describe 'ceilometer::collector' do
:name => platform_params[:collector_service_name],
:enable => false,
:hasstatus => true,
:hasrestart => true
:hasrestart => true,
:tag => 'ceilometer-service'
)
end
end
@ -104,7 +106,8 @@ describe 'ceilometer::collector' do
:name => platform_params[:collector_service_name],
:enable => false,
:hasstatus => true,
:hasrestart => true
:hasrestart => true,
:tag => 'ceilometer-service'
)
end
end