fix(logs): do not add exporters mutliple times to the pipeline (#3796)

* fix(logs): do not add exporters mutliple times to the pipeline Signed-off-by: Dominik Rosiek <[email protected]> * chore: tests Signed-off-by: Dominik Rosiek <[email protected]> * chore: changelog Signed-off-by: Dominik Rosiek <[email protected]> --------- Signed-off-by: Dominik Rosiek <[email protected]>
SumoLogic · Jul 5, 2024 · a5d4713 · a5d4713
1 parent e52a34c
commit a5d4713
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 0 deletions.
diff --git a/.changelog/3796.fixed.txt b/.changelog/3796.fixed.txt
@@ -0,0 +1 @@
+fix(logs): do not add exporters mutliple times to the pipeline
diff --git a/deploy/helm/sumologic/templates/_helpers/_logs.tpl b/deploy/helm/sumologic/templates/_helpers/_logs.tpl
@@ -127,6 +127,7 @@ Return all exporters for .Type pipeline
 {{- $exporters = append $exporters $exporter -}}
 {{- end -}}
 {{- end -}}
+{{- $exporters = uniq $exporters -}}
 {{- range $_, $exporter := $exporters }}
 {{ printf "- %s" $exporter }}
 {{- end }}

diff --git a/tests/helm/testdata/goldenfile/metadata_logs_otc/routing.input.yaml b/tests/helm/testdata/goldenfile/metadata_logs_otc/routing.input.yaml
@@ -13,6 +13,7 @@ sumologic:
       routing:
         fallbackExporters:
           - debug
+          - sumologic/test
         table:
           - exporter: sumologic/test
             statement: route() where resource.attributes["exporter"] == "test"
diff --git a/tests/helm/testdata/goldenfile/metadata_logs_otc/routing.output.yaml b/tests/helm/testdata/goldenfile/metadata_logs_otc/routing.output.yaml
@@ -221,13 +221,15 @@ data:
       routing/containers:
         default_exporters:
         - debug
+        - sumologic/test
         table:
         - exporters:
           - sumologic/test
           statement: route() where resource.attributes["exporter"] == "test"
       routing/systemd:
         default_exporters:
         - debug
+        - sumologic/test
         table:
         - exporters:
           - sumologic/test
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		fix(logs): do not add exporters mutliple times to the pipeline