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

Move metric requirement level definition from spec to semconv repo #822

Closed
lmolkova opened this issue Mar 18, 2024 · 2 comments · Fixed by #831
Closed

Move metric requirement level definition from spec to semconv repo #822

lmolkova opened this issue Mar 18, 2024 · 2 comments · Fixed by #831
Assignees

Comments

@lmolkova
Copy link
Contributor

lmolkova commented Mar 18, 2024

Metrics requirement levels makes sense in a scope of semantic convention and the corresponding doc https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/metric-requirement-level.md should belong in this repo.

Note: attribute requirement levels live in the semconv - https://github.com/open-telemetry/semantic-conventions/blob/0220d78f63e088a50fe8e4f827b97a28a42aeb23/docs/general/attribute-requirement-level.md

@joaopgrassi
Copy link
Member

I guess we will have to modify all conventions after the move, but that's doable with a find/replace all :). I can fix it if you are not already on it.

@lmolkova
Copy link
Contributor Author

I didn't get to it, so please go ahead if you can. Thanks!

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