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

Adding @reta to OpenSearch maintainers. #2905

Merged
merged 2 commits into from
Apr 15, 2022

Conversation

dblock
Copy link
Member

@dblock dblock commented Apr 14, 2022

Signed-off-by: dblock dblock@amazon.com

Description

Following the process in opensearch-project/.github#59, I had nominated Andriy Redko (@reta) as a co-maintainer. The maintainers have voted and agreed to this nomination.

Andriy is the author of 124 PRs into OpenSearch, second in code change volume since the fork.

image

Some of reta@’s notable early code contributions were in the supporting infrastructure space, and have increasingly been shifting into adding new features:

Andriy is active almost daily in the project, thoughtfully reviewing and commenting on pull requests (example). He also has write permissions on https://github.com/opensearch-project/opensearch-plugin-template-java/ already.

This PR also sorts current maintainers alphabetically.

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: dblock <dblock@amazon.com>
@dblock dblock requested a review from a team as a code owner April 14, 2022 18:38
Copy link
Collaborator

@VachaShah VachaShah left a comment

Choose a reason for hiding this comment

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

This is awesome!

Copy link
Member

@owaiskazi19 owaiskazi19 left a comment

Choose a reason for hiding this comment

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

Welcome to the club, @reta!

Copy link
Member

@kartg kartg left a comment

Choose a reason for hiding this comment

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

💯

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 6cb31d6
Log 4487

Reports 4487

MAINTAINERS.md Outdated Show resolved Hide resolved
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Copy link
Collaborator

@nknize nknize left a comment

Choose a reason for hiding this comment

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

LGTM

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success b431ceb
Log 4491

Reports 4491

@reta
Copy link
Collaborator

reta commented Apr 14, 2022

So proud to be nominated, thank you so much team!! ❤️ ❤️ ❤️ ❤️ I will continue to do my best but it is nearly impossible to chase @nknize 😄 😄

@mattweber
Copy link
Contributor

Congrats @reta!

@dblock
Copy link
Member Author

dblock commented Apr 15, 2022

Screen Shot 2022-04-15 at 11 10 00 AM

@dblock dblock merged commit a6b4967 into opensearch-project:main Apr 15, 2022
@dblock dblock deleted the maintainer-reta branch April 15, 2022 15:10
@nknize
Copy link
Collaborator

nknize commented Apr 15, 2022

So happy to have you, @reta, as a maintainer! Some excellent contributions over the last months. Thanks for sticking around while the maintainer process was being figured out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.