]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Update default event publisher
authorPradeep Kilambi <pkilambi@redhat.com>
Thu, 6 Apr 2017 21:45:12 +0000 (17:45 -0400)
committerPradeep Kilambi <pkilambi@redhat.com>
Thu, 6 Apr 2017 21:53:33 +0000 (17:53 -0400)
In pike, the default publisher is gnocchi. Lets use that
as the default. If you are using collector the override
this to notifier:// instead.

Change-Id: Ic6ba909c873bee801843718cc25974abeda9f857

manifests/agent/notification.pp
releasenotes/notes/update-default-event-publisher-623aff98f426e60f.yaml [new file with mode: 0644]
spec/classes/ceilometer_agent_notification_spec.rb

index 6e95526e85263dd1b281b6b7d5a491973cd92de1..b1fccd13b4c25b9e8809e85205fc48c5e59f6e39 100644 (file)
@@ -65,7 +65,7 @@
 #   (Optional) A list of publishers to put in event_pipeline.yaml
 #   Add 'notifier://?topic=alarm.all' to the list if you are using Aodh
 #   for alarms.
-#   Defaults to ['notifier://'],
+#   Defaults to ['gnocchi://'],
 #
 class ceilometer::agent::notification (
   $manage_service            = true,
@@ -77,7 +77,7 @@ class ceilometer::agent::notification (
   $messaging_urls            = $::os_service_default,
   $package_ensure            = 'present',
   $manage_event_pipeline     = false,
-  $event_pipeline_publishers = ['notifier://'],
+  $event_pipeline_publishers = ['gnocchi://'],
 ) {
 
   include ::ceilometer::deps
diff --git a/releasenotes/notes/update-default-event-publisher-623aff98f426e60f.yaml b/releasenotes/notes/update-default-event-publisher-623aff98f426e60f.yaml
new file mode 100644 (file)
index 0000000..d2d6221
--- /dev/null
@@ -0,0 +1,4 @@
+---
+fixes:
+  - Update the default event publisher to be gnocchi://. This is the
+    default upstream.
index 0c0d905dba1e87ebad0e9d1f0f9c3b4b566172dd..03b5a49917bcbae30e659a07ac1c3cb53ac9c1cc 100644 (file)
@@ -136,7 +136,7 @@ describe 'ceilometer::agent::notification' do
           "      transformers:",
           "      triggers:",
           "      publishers:",
-          "          - notifier://",
+          "          - gnocchi://",
       ])}
     end