Skip to content

Add Prometheus interoperability to codeowners#5027

Open
ArthurSens wants to merge 1 commit intoopen-telemetry:mainfrom
ArthurSens:prom-codeowners
Open

Add Prometheus interoperability to codeowners#5027
ArthurSens wants to merge 1 commit intoopen-telemetry:mainfrom
ArthurSens:prom-codeowners

Conversation

@ArthurSens
Copy link
Copy Markdown
Member

@ArthurSens ArthurSens commented Apr 14, 2026

Changes

@johannaojeling just let me know that she opened a few PRs for the Prometheus spec, and they went unnoticed. If the TC is ok with it, I'm adding the @open-telemetry/prometheus-interoperability as codeowners of the two Prometheus spec files, to ensure we get notified when PRs against these files are open.

If this is too much, I don't have a problem closing the PR :)

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@ArthurSens ArthurSens requested review from a team as code owners April 14, 2026 13:07
@dashpole
Copy link
Copy Markdown
Contributor

If we do this we should clean up the interoperability group while we are at it

@ArthurSens ArthurSens added the Skip Changelog Label to skip the changelog check label Apr 15, 2026
Comment thread .github/CODEOWNERS
Comment thread .github/CODEOWNERS

# Prometheus-specific specifications
/specification/compatibility/prometheus_and_openmetrics.md @open-telemetry/technical-committee @open-telemetry/spec-sponsors @open-telemetry/prometheus-interoperability
/specification/metrics/sdk_exporters/prometheus.md @open-telemetry/technical-committee @open-telemetry/spec-sponsors @open-telemetry/prometheus-interoperability
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't just notify the group, it adds the group as official approvers who's approval counters towards the merge requirements for PRs.

I think this is a good idea, but first, should do some housekeeping and make sure the team membership reflects people actively involved in the interoperability effort, as @dashpole suggests: #5027 (comment)

Copy link
Copy Markdown
Contributor

@jmacd jmacd left a comment

Choose a reason for hiding this comment

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

Thank you @ArthurSens!

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

None yet

Development

Successfully merging this pull request may close these issues.

6 participants