-
Notifications
You must be signed in to change notification settings - Fork 896
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
Refine SDK MeterProvider configuration section #3522
Conversation
The tracing spec has something like this /~https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk.md#tracer-creation. We should consider consistency (might be better to change both). |
I created an issue (feel free to assign it me) I want to make this PR easier (baby steps). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saw you created an issue for applying this change to tracing. Can you also create an issue to track making this same change to the log SDK?
Please consider merging the PR 😉 |
@pellared can you add a changelog entry? |
What do you want me to write there?
? |
Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>
Towards open-telemetry/opentelemetry-go#3642 (comment)
Changes
MeterProvider - Configuration
section.