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

Promote @jaronoff97 to maintainer #1419

Merged
merged 3 commits into from
Feb 6, 2023
Merged

Promote @jaronoff97 to maintainer #1419

merged 3 commits into from
Feb 6, 2023

Conversation

jaronoff97
Copy link
Contributor

@jaronoff97 jaronoff97 commented Feb 1, 2023

Hello!

I've been very active in the operator project and have taken a greater role in the project with the merging of #1339 and #1326. In addition to that, I've been part of leading the work for the target allocator and we've seen great adoption of it as a result of our efforts. You can see the list of issues and PRs i've submitted. I've also been active in responding to people in our community slack for support. I would love to help out in a greater capacity.

cc @open-telemetry/operator-maintainers

@jaronoff97 jaronoff97 requested a review from a team February 1, 2023 15:25
@jaronoff97 jaronoff97 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Feb 1, 2023
@pavolloffay
Copy link
Member

Role definition and requirements https://github.com/open-telemetry/community/blob/main/community-membership.md#maintainer

I support @jaronoff97 nomination. He is doing great work, by helping to maintain the project and build new capabilities in the target alocator and OpAMP

Copy link
Member

@pavolloffay pavolloffay left a comment

Choose a reason for hiding this comment

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

I support the nomination.

LGTM, just one nit.

README.md Outdated
Maintainers ([@open-telemetry/operator-maintainers](https://github.com/orgs/open-telemetry/teams/operator-maintainers)):

- [Juraci Paixão Kröhling](https://github.com/jpkrohling), Grafana Labs
- [Pavol Loffay](https://github.com/pavolloffay), Red Hat
- [Vineeth Pothulapati](https://github.com/VineethReddy02), Timescale
- [Jacob Aronoff](https://github.com/jaronoff97), Lightstep
Copy link
Member

Choose a reason for hiding this comment

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

we should probably alphabetically sort these.

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.

As a not-maintainer my vote doesn't count, but I vote yes anyway!

Copy link
Member

@frzifus frzifus left a comment

Choose a reason for hiding this comment

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

Same here :)

@pavolloffay
Copy link
Member

@jpkrohling @VineethReddy02 could you please vote?

@jpkrohling
Copy link
Member

I support this nomination!

@VineethReddy02
Copy link
Contributor

I support the nomination, LGTM!

@pavolloffay
Copy link
Member

Merging, welcome @jaronoff97!! 🎉

@pavolloffay pavolloffay merged commit 4dfbab9 into main Feb 6, 2023
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* Promote @jaronoff97 to maintainer

* order, name (open-telemetry#1420)

* Alphabetically sort (open-telemetry#1432)
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.

6 participants