Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

autoexport: Accept a comma-separated list to enable setting multiple exporters #4471

Open
pellared opened this issue Oct 23, 2023 · 1 comment · May be fixed by #5830
Open

autoexport: Accept a comma-separated list to enable setting multiple exporters #4471

pellared opened this issue Oct 23, 2023 · 1 comment · May be fixed by #5830
Assignees
@thomasgouveia
Labels
enhancement New feature or request exporter: autoexport

Comments

@pellared
Copy link
Member

From https://github.com/open-telemetry/opentelemetry-specification/blob/563958cb2bd8529990f19fdce7a5f3643bf63091/specification/configuration/sdk-environment-variables.md?plain=1#L222:

The SDK MAY accept a comma-separated list to enable setting multiple exporters.

This would be useful, but doesn't really fit with how the package is currently designed.

Originally posted by @dashpole in #4229 (comment)
@pellared pellared added enhancement New feature or request exporter: autoexport labels Oct 23, 2023
@pellared pellared mentioned this issue Oct 23, 2023
Merged
@thomasgouveia
Copy link
Contributor

thomasgouveia commented Jun 28, 2024 •
edited
Loading

@pellared, it's something I can work on. Could you assign it to me? I'll make a proposal here to get feedback before proceeding with the implementation.

thomasgouveia added a commit to thomasgouveia/opentelemetry-go-contrib that referenced this issue Jun 30, 2024
@thomasgouveia
feat(exporters/autoexport): enable support of multiple exporters for …
7167054 
…OTEL_*_EXPORTER (open-telemetry#4471)

This commit introduces the support of comma-separated value for OTEL_{METRICS,TRACES,LOGS}_EXPORTER.
New functions can now be used to intialize a list of exporters: NewMetricReaders, NewLogExporters, NewSpanExporters.

Old ones (NewMetricReader, NewLogExporter, NewSpanExporter) are now deprecated but still continue to do they initial work to avoid breaking change.

Signed-off-by: thomasgouveia <[email protected]>
thomasgouveia added a commit to thomasgouveia/opentelemetry-go-contrib that referenced this issue Jun 30, 2024
@thomasgouveia
feat(exporters/autoexport): enable support of multiple exporters for …
464b8be 
…OTEL_*_EXPORTER (open-telemetry#4471)

This commit introduces the support of comma-separated value for OTEL_{METRICS,TRACES,LOGS}_EXPORTER.
New functions can now be used to intialize a list of exporters: NewMetricReaders, NewLogExporters, NewSpanExporters.

Old ones (NewMetricReader, NewLogExporter, NewSpanExporter) are now deprecated but still continue to do they initial work to avoid breaking change.

Signed-off-by: thomasgouveia <[email protected]>
thomasgouveia added a commit to thomasgouveia/opentelemetry-go-contrib that referenced this issue Jun 30, 2024
@thomasgouveia
feat(exporters/autoexport): enable support of multiple exporters for …
7f882f1 
…OTEL_*_EXPORTER (open-telemetry#4471)

This commit introduces the support of comma-separated value for OTEL_{METRICS,TRACES,LOGS}_EXPORTER.
New functions can now be used to intialize a list of exporters: NewMetricReaders, NewLogExporters, NewSpanExporters.

Old ones (NewMetricReader, NewLogExporter, NewSpanExporter) are now deprecated but still continue to do they initial work to avoid breaking change.

Signed-off-by: thomasgouveia <[email protected]>
@thomasgouveia thomasgouveia linked a pull request Jul 1, 2024 that will close this issue
Open
thomasgouveia added a commit to thomasgouveia/opentelemetry-go-contrib that referenced this issue Jul 7, 2024
@thomasgouveia
feat(exporters/autoexport): enable support of multiple exporters for …
1a6327a 
…OTEL_*_EXPORTER (open-telemetry#4471)

This commit introduces the support of comma-separated value for OTEL_{METRICS,TRACES,LOGS}_EXPORTER.
New functions can now be used to intialize a list of exporters: NewMetricReaders, NewLogExporters, NewSpanExporters.

Old ones (NewMetricReader, NewLogExporter, NewSpanExporter) are now deprecated but still continue to do they initial work to avoid breaking change.

Signed-off-by: thomasgouveia <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thomasgouveia thomasgouveia

Labels
enhancement New feature or request exporter: autoexport
Projects
None yet
Milestone
No milestone
2 participants
@pellared @thomasgouveia