openstack/puppet-ceilometer
Code Issues Proposed changes

Merge "Update default event publisher"

Browse Source
This commit is contained in:
Jenkins 2017-04-10 15:58:45 +00:00 committed by Gerrit Code Review
commit bd1ebe015f
3 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/agent/notification.pp
View File

@ -65,7 +65,7 @@
# (Optional) A list of publishers to put in event_pipeline.yaml # (Optional) A list of publishers to put in event_pipeline.yaml
# Add 'notifier://?topic=alarm.all' to the list if you are using Aodh # Add 'notifier://?topic=alarm.all' to the list if you are using Aodh
# for alarms. # for alarms.
# Defaults to ['notifier://'], # Defaults to ['gnocchi://'],
# #
class ceilometer::agent::notification ( class ceilometer::agent::notification (
$manage_service = true, $manage_service = true,
@ -77,7 +77,7 @@ class ceilometer::agent::notification (
$messaging_urls = $::os_service_default, $messaging_urls = $::os_service_default,
$package_ensure = 'present', $package_ensure = 'present',
$manage_event_pipeline = false, $manage_event_pipeline = false,
$event_pipeline_publishers = ['notifier://'], $event_pipeline_publishers = ['gnocchi://'],
) { ) {
include ::ceilometer::deps include ::ceilometer::deps

4
releasenotes/notes/update-default-event-publisher-623aff98f426e60f.yaml Normal file
View File

@ -0,0 +1,4 @@
---
fixes:
- Update the default event publisher to be gnocchi://. This is the
default upstream.

2
spec/classes/ceilometer_agent_notification_spec.rb
View File

@ -136,7 +136,7 @@ describe 'ceilometer::agent::notification' do
" transformers:", " transformers:",
" triggers:", " triggers:",
" publishers:", " publishers:",
" - notifier://", " - gnocchi://",
])} ])}
end end