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

[chore] remove codeowners who didn't become members after a period of wait #36229

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Nov 6, 2024

Alternative or first step to #36227

This PR removes codeowners who didn't become members after a long wait.

Fixes #30552

@atoulme atoulme force-pushed the remove_old_allowlist_members branch from 03b9724 to e75f2e5 Compare November 9, 2024 08:34
@atoulme atoulme force-pushed the remove_old_allowlist_members branch from e75f2e5 to 60e9a26 Compare November 9, 2024 08:34
@atoulme
Copy link
Contributor Author

atoulme commented Nov 9, 2024

Rebased to fix conflicts ; also keeping @emreyalvac since he has started working on his membership request.

@TylerHelmuth
Copy link
Member

@open-telemetry/collector-contrib-approvers i plan to merge this by end-of-week unless we get objections. Will merge sooner if we get enough approvals.

@songy23
Copy link
Member

songy23 commented Nov 12, 2024

Indeed there are some issues with allowlist on mainline: https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/11801390829/job/32874673060

could you fix as well?

Copy link
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

@dmitryax dmitryax merged commit 53402aa into open-telemetry:main Nov 12, 2024
158 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 12, 2024
@asaharn
Copy link
Member

asaharn commented Nov 19, 2024

Hi @atoulme @mx-psi, I think there is some confusion. I am still an active contributor along with @ag-ramachandran for ADX Exporter component. May I know how can I become a member to fix this?

@TylerHelmuth
Copy link
Member

@asaharn @ag-ramachandran follow the instructions here. I'll be one of your sponsors.

@mx-psi
Copy link
Member

mx-psi commented Nov 19, 2024

I am happy to be another of the sponsors for you

@asaharn
Copy link
Member

asaharn commented Nov 19, 2024

Thank you so much @TylerHelmuth @mx-psi. I will raise the ticket and update you.

@asaharn
Copy link
Member

asaharn commented Nov 19, 2024

Here is the ticket open-telemetry/community#2446

TylerHelmuth added a commit that referenced this pull request Dec 12, 2024
#### Description

This exporter was added as a vendor-specific component via
#24294.
It [no longer has its vendor sponsor as a code
owner](#36229).

Following our [Unmaintained
guidelines](https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/component-stability.md#unmaintained),
this component is now considered unmaintained.

@am-kinetica @kyle-sutton please respond if Kinetica is interested in
continuing to maintain this component.
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
… wait (open-telemetry#36229)

Alternative or first step to open-telemetry#36227 

This PR removes codeowners who didn't become members after a long wait.

Fixes open-telemetry#30552
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
#### Description

This exporter was added as a vendor-specific component via
open-telemetry#24294.
It [no longer has its vendor sponsor as a code
owner](open-telemetry#36229).

Following our [Unmaintained
guidelines](https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/component-stability.md#unmaintained),
this component is now considered unmaintained.

@am-kinetica @kyle-sutton please respond if Kinetica is interested in
continuing to maintain this component.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure that all codeowners are members of the OpenTelemetry organization
8 participants