Skip to content

CODEOWNERS: remove @braydonk and @jefferbrecht - #11909

Merged
edsiper merged 1 commit into
fluent:masterfrom
braydonk:remove_braydonk_jefferbrecht
Jun 3, 2026
Merged

CODEOWNERS: remove @braydonk and @jefferbrecht#11909
edsiper merged 1 commit into
fluent:masterfrom
braydonk:remove_braydonk_jefferbrecht

Conversation

@braydonk

@braydonk braydonk commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Going forward, @JeffLuoo will be the primary owner for the out_stackdriver plugin. I am removing myself and @jefferbrecht from any code owner status. Jeff Luo is now fully in charge of the plugin, along with anyone he may want to add as code owners in the future.

I do still have maintainer privileges on this repo, which was simply so I could merge PRs that only touched the out_stackdriver plugin/test code without needing to bother maintainers. I can't do it myself, but I would suggest this permission and general interaction be passed on to @JeffLuoo as well.

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.

Summary by CodeRabbit

  • Chores
    • Updated code ownership assignments for the Stackdriver output plugin and its runtime test data to reflect current maintainers. This clarifies maintenance and review responsibilities. No functional, public API, or exported-entity changes were made. Low review effort; minor metadata-only edits.

@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

CODEOWNERS updated to consolidate responsibility for the Stackdriver output plugin and its runtime test data to a single owner: @jeffluoo.

Changes

Ownership Update

Layer / File(s) Summary
Stackdriver plugin and test ownership
CODEOWNERS
Plugin entry /plugins/out_stackdriver and test file entries /tests/runtime/out_stackdriver.c and /tests/runtime/data/stackdriver are reassigned to single owner @jeffluoo.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • edsiper

Poem

🐰 I hopped through CODEOWNERS on a spree,
One name now tends the Stackdriver tree,
Jeff takes the keys and polishes the light,
Plugins and tests sleep cozy at night,
Hooray for tidy fences, snug and right!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title directly and accurately summarizes the main change—removing two code owners from the CODEOWNERS file.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@braydonk
braydonk force-pushed the remove_braydonk_jefferbrecht branch from 09ff41a to 037199f Compare June 3, 2026 13:52
Going forward, @JeffLuoo will be the primary owner for the
`out_stackdriver` plugin. I am removing myself an @jefferbrecht from any
code owner status. Jeff Luo is now fully in charge of the plugin, along
with anyone he may want to add as code owners in the future.

I do still have maintainer privileges on this repo, which was simply so
I could merge PRs that only touched the `out_stackdriver` plugin/test
code without needing to bother maintainers. I can't do it myself, but I
would suggest this permission and general interaction be passed on to
@JeffLuoo as well.

Signed-off-by: braydonk <braydonk@google.com>
@braydonk
braydonk force-pushed the remove_braydonk_jefferbrecht branch from 037199f to f577f6e Compare June 3, 2026 13:52
@braydonk braydonk changed the title [chore] CODEOWNERS: remove @braydonk and @jefferbrecht CODEOWNERS: remove @braydonk and @jefferbrecht Jun 3, 2026
@edsiper
edsiper merged commit 7bb00ae into fluent:master Jun 3, 2026
29 of 37 checks passed
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.

4 participants