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

[processor/{deltatorate, metricsgeneration}] Add CODEOWNERS #15375

Merged
merged 4 commits into from
Oct 22, 2022

Conversation

mx-psi
Copy link
Member

@mx-psi mx-psi commented Oct 21, 2022

Description:

Add CODEOWNERs for the deltatorate and metricsgeneration processors, based on the authors of #4218 and #3266 respectively.

Link to tracking Issue: Updates #3870

@mx-psi mx-psi added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Oct 21, 2022
@mx-psi mx-psi requested a review from Aneurysm9 October 21, 2022 10:23
@mx-psi mx-psi requested a review from a team as a code owner October 21, 2022 10:23
@mx-psi mx-psi changed the title [processor/deltatorate] Add CODEOWNERS [processor/{deltatorate, metricsgeneration}] Add CODEOWNERS Oct 21, 2022
Copy link
Member

@Aneurysm9 Aneurysm9 left a comment

Choose a reason for hiding this comment

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

I can't speak for @hossain-rayhan.

Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

As much as I'd like code owners on everything, I do think we should adhere to the guidelines we setup for ourselves, which require code owners to be OpenTelemetry community members.

@mx-psi
Copy link
Member Author

mx-psi commented Oct 21, 2022

I hadn't realized that Rayhan was not a member, PTAL

@mx-psi
Copy link
Member Author

mx-psi commented Oct 21, 2022

Also @Aneurysm9 please then confirm that you would be okay being the single owner of these components

Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

@bogdandrutu bogdandrutu merged commit b03e272 into open-telemetry:main Oct 22, 2022
shalper2 pushed a commit to shalper2/opentelemetry-collector-contrib that referenced this pull request Dec 6, 2022
…emetry#15375)

* [processor/deltatorate] Add CODEOWNERS

* Also add `metricsgeneration`

* Update ALLOWLIST

* Remove @hossain-rayan
@plantfansam plantfansam mentioned this pull request Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants