]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Remove unused "transformers" field from event pipeline configuration
authorTakashi Kajinami <tkajinam@redhat.com>
Thu, 24 Mar 2022 16:43:07 +0000 (01:43 +0900)
committerTakashi Kajinami <tkajinam@redhat.com>
Thu, 24 Mar 2022 23:16:38 +0000 (08:16 +0900)
Transformer was already removed from ceilometer by [1]. This field
was removed from the example configuration file when transformer was
deprecated[2].

[1] 9db5c6c9bfc66018aeb78c4a262e1bfa9b326798
[2] 1dcbd607df0696101b40f77d7721489679ebe0ba

Change-Id: I012d3f960d2a562c2735e7b5c1e408a3a1addb70

spec/classes/ceilometer_agent_notification_spec.rb
templates/event_pipeline.yaml.erb

index 4872009784464cf9af4a8940961b7e73c2589b41..16bc8112b4e102898537630a15604b1a5a92ffa7 100644 (file)
@@ -27,8 +27,8 @@ describe 'ceilometer::agent::notification' do
   end
 
   let :params do
-    { :manage_service     => true,
-      :enabled            => true }
+    { :manage_service => true,
+      :enabled        => true }
   end
 
   shared_examples_for 'ceilometer-agent-notification' do
@@ -125,7 +125,6 @@ describe 'ceilometer::agent::notification' do
           "          - event_sink",
           "sinks:",
           "    - name: event_sink",
-          "      transformers:",
           "      publishers:",
           "          - gnocchi://",
       ])}
@@ -148,7 +147,6 @@ describe 'ceilometer::agent::notification' do
           "          - event_sink",
           "sinks:",
           "    - name: event_sink",
-          "      transformers:",
           "      publishers:",
           "          - notifier://",
           "          - notifier://?topic=alarm.all",
@@ -165,9 +163,8 @@ describe 'ceilometer::agent::notification' do
             'sinks'  => ['my_event_sink'],
           ],
           'sinks'   => [
-            'name'         => 'my_event_sink',
-            'transformers' => [],
-            'publishers'   => ['gnocchi://'],
+            'name'       => 'my_event_sink',
+            'publishers' => ['gnocchi://'],
           ],
         }
       )}
@@ -182,7 +179,6 @@ sources:
   - my_event_sink
 sinks:
 - name: my_event_sink
-  transformers: []
   publishers:
   - gnocchi://
 ',
@@ -219,9 +215,8 @@ sinks:
             'sinks'  => ['my_sink'],
           ],
           'sinks'   => [
-            'name'         => 'my_sink',
-            'transformers' => [],
-            'publishers'   => ['gnocchi://'],
+            'name'       => 'my_sink',
+            'publishers' => ['gnocchi://'],
           ],
         }
       )}
@@ -236,7 +231,6 @@ sources:
   - my_sink
 sinks:
 - name: my_sink
-  transformers: []
   publishers:
   - gnocchi://
 ',
index 9a46fbfc322e1f092a5c149524f89b11b7e1ba88..39c2f664a07f7803990ecdbecb99370b1392606b 100644 (file)
@@ -7,7 +7,6 @@ sources:
           - event_sink
 sinks:
     - name: event_sink
-      transformers:
       publishers:
 <% @event_pipeline_publishers.each do |publisher| -%>
           - <%= publisher %>