Skip to content

Add core semconv area#3309

Merged
lmolkova merged 1 commit intoopen-telemetry:mainfrom
lmolkova:add-core-to-areas
Jan 20, 2026
Merged

Add core semconv area#3309
lmolkova merged 1 commit intoopen-telemetry:mainfrom
lmolkova:add-core-to-areas

Conversation

@lmolkova
Copy link
Copy Markdown
Member

@lmolkova lmolkova commented Jan 19, 2026

Fixes #3306
See also #3308, #3310

Put areas listed in https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/semantic-conventions.md (that spec relies on) under the ownership of @open-telemetry/specs-semconv-approvers

@lmolkova lmolkova requested review from a team as code owners January 19, 2026 19:10
@lmolkova lmolkova added the Skip Changelog Label to skip the changelog check label Jan 19, 2026
Copy link
Copy Markdown
Contributor

@jsuereth jsuereth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This causes some overlap between, e.g. service and the Service + Deployment SIG (I think also for server).

I believe this is fine, core semconv approvers / maintainers should be able to comment on these as they have specification implications. So implicitly they've had that repsonsibility. I'm on board.

@jsuereth jsuereth moved this from Untriaged to Needs More Approval in Semantic Conventions Triage Jan 20, 2026
@lmolkova
Copy link
Copy Markdown
Member Author

overlap is fine (same as overlap in codeowners), this ensures that even when service and deployment SIG has achieved its goals and is dissolved, we have ownership for core/required components and PRs against them are not automatically closed.

@lmolkova lmolkova added this pull request to the merge queue Jan 20, 2026
Merged via the queue into open-telemetry:main with commit f1c814f Jan 20, 2026
16 of 17 checks passed
@lmolkova lmolkova deleted the add-core-to-areas branch January 20, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog Label to skip the changelog check

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Request owner for error area (blocked semconv PR adding error.source)

3 participants