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

Stabilize opentelemetry-semconv #90

Open
jack-berg opened this issue Sep 25, 2024 · 2 comments
Open

Stabilize opentelemetry-semconv #90

jack-berg opened this issue Sep 25, 2024 · 2 comments

Comments

@jack-berg
Copy link
Member

Want to start a conversation about what stands in the way of marking the opentelemetry-semconv artifact as stable. We've come a long way recently, re-organizing by domain, switching from build-tools to weaver based code generation, and changing how we think about deprecated / incubating attributes.

@breedx-splk
Copy link

With incubating in place, it seems like most of the roadblocks have been cleared. I suppose the main thing to think about is needing to be able to support future breaking changes though major semver bumps, which should be seldom and annoying at worst?

@jack-berg
Copy link
Member Author

It would also be good to think through what might change in a future where we're generating helpers / constants for producing semantic convention metrics (#12), events, and anything else.

Does the class organization around root namespace work for this? For example, we currently have classes named HttpAttributes and HttpIncubatingAttributes. These names imply that the only hold the http attributes. If we added metric generation, would we feel comfortable adding classes like HttpMetrics, HttpIncubatingMetrics?

Maybe we think of the current class organization as the embodiment of the semantic convention attribute registry - a global registry of attributes agnostic of which conventions they're used in. Then, we can independently decide how to organize anything else we generate for specific conventions.

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

No branches or pull requests

2 participants