-
Notifications
You must be signed in to change notification settings - Fork 440
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
Conversation
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 |
There was a problem hiding this 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 |
There was a problem hiding this comment.
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.
There was a problem hiding this 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!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here :)
@jpkrohling @VineethReddy02 could you please vote? |
I support this nomination! |
I support the nomination, LGTM! |
Merging, welcome @jaronoff97!! 🎉 |
* Promote @jaronoff97 to maintainer * order, name (open-telemetry#1420) * Alphabetically sort (open-telemetry#1432)
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