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

CODEOWNERS: change out_stackdriver code owners #9806

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

braydonk
Copy link
Contributor

@braydonk braydonk commented Jan 7, 2025

Due to some changes in the team at Google, we have decided to adjust the codeowners for the out_stackdriver plugin. Since @qingling128 and @igorpeshansky have transitioned to new teams not involved with the plugin, we have decided to remove them from CODEOWNERS so they don't need to deal with the context switching it would require. In their place we are adding @jefferbrecht and @JeffLuoo, who have both been working with Fluent Bit as long or longer than I have and have lots of necessary context to be able to review out_stackdriver PRs. We are hoping to add more in the future.

Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

Due to some changes in the team, we have decided to adjust the
codeowners for the `out_stackdriver` plugin. Since @lingshi and
@igorpeshansky have transitioned to new teams not involved with the
plugin, we have decided to remove them from CODEOWNERS so they don't
need to deal with the context switching it would require. In their place
we are adding @jefferbrecht, who has been working with Fluent Bit around
as long as I have and has lots of necessary context to be able to review
out_stackdriver PRs. We are hoping to add more in the future.

Signed-off-by: braydonk <[email protected]>
@braydonk braydonk force-pushed the sd_codeowners_update branch from 64b5227 to eeb7b6f Compare January 7, 2025 22:09
@edsiper edsiper merged commit c62b427 into fluent:master Jan 8, 2025
37 of 38 checks passed
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.

3 participants