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.

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

autoconfigure #2320

Open
hdost opened this issue May 31, 2022 · 4 comments
Open

autoconfigure #2320

hdost opened this issue May 31, 2022 · 4 comments
Labels

Comments

@hdost
Copy link

hdost commented May 31, 2022

Problem Statement

There exists https://github.com/open-telemetry/opentelemetry-java/tree/main/sdk-extensions/autoconfigure for Java. Is this something that would be of interest in golang? Would it potentially go into go-contrib?

Proposed Solution

Implement autoconfig

Alternatives

Don't do it.

Prior Art

Already linked Java

@hdost hdost added the enhancement New feature or request label May 31, 2022
@dmathieu
Copy link
Member

We already support environment variables for configuration, as specified.

@hdost
Copy link
Author

hdost commented May 31, 2022

The functionality I'm specifically interested in is Exporter Selection
This does not appear to be implemented https://sourcegraph.com/search?q=context:global+repo:%5Egithub%5C.com/open-telemetry/opentelemetry-go+OTEL_TRACES_EXPORTER&patternType=literal

@Aneurysm9
Copy link
Member

We have discussed using a registry method similar to that proposed in #2258 for exporters as well. There are some issues to resolve regarding registration order and safety and we expect that any initial implementation will live in the contrib repo to give it room to explore the solution space.

@Aneurysm9 Aneurysm9 transferred this issue from open-telemetry/opentelemetry-go May 31, 2022
@deejgregor
Copy link

See #2591 / #2592. I think this might cover what you are thinking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants