Skip to content

rewrite presets.coralogixExporter.pipeline from string to array to allow enabling exporter on 2 pipelines at once#234

Merged
douglascamata merged 5 commits into
mainfrom
cds-2254-exporter-pipeline-type
Jul 2, 2025
Merged

rewrite presets.coralogixExporter.pipeline from string to array to allow enabling exporter on 2 pipelines at once#234
douglascamata merged 5 commits into
mainfrom
cds-2254-exporter-pipeline-type

Conversation

@cazorla19

@cazorla19 cazorla19 commented Jul 2, 2025

Copy link
Copy Markdown

Fixes CDS-2254

This change will add new variable presets.coralogixExporter.pipelines as an array of string to allow enabling exporter on 2 pipelines at once.

So that in cases where we want to disable, let's say, coralogix exporter only in one pipeline, we could do this:

opentelemetry-agent:
  presets:
    coralogixExporter:
      enabled: true
      pipelines: ["metrics", "traces"]

instead of this:

opentelemetry-agent:
  presets:
    coralogixExporter:
      enabled: true
      pipeline: "none"
  config:
    service:
      pipelines:
        metrics:
          exporters:
            - coralogix
        traces:
          exporters:
            - coralogix

We've added a new variable instead of changing the existing one (presets.coralogixExporter.pipeline) for backward compatibility. The existing one pipeline will be deprecated in the future. For now, pipelineS will take precedence, while pipeline will be converted into array when set.

…rray to allow enabling exporter on 2 pipelines at once
@cazorla19 cazorla19 requested review from a team, nicolastakashi and oded-dd as code owners July 2, 2025 12:43
Comment thread charts/opentelemetry-collector/CHANGELOG.md Outdated

@douglascamata douglascamata left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job! I have two small suggestions.

Comment thread charts/opentelemetry-collector/templates/NOTES.txt
Comment thread charts/opentelemetry-collector/templates/_config.tpl Outdated
@douglascamata

Copy link
Copy Markdown
Member

Something else, @cazorla19: can you please add one example for each of the options to helps us see how they behave? I think we might already have one with pipeline, so just missing the pipelines one.

@douglascamata douglascamata merged commit 7327f83 into main Jul 2, 2025
3 checks passed
@douglascamata douglascamata deleted the cds-2254-exporter-pipeline-type branch July 2, 2025 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants