You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
maintainers: update maintainers list and code owners - #12785
MAINTAINERS.md and CODEOWNERS no longer reflect who maintains the
project today:
Several code owner entries point at handles without write access to
the repository, so GitHub reports those lines as invalid.
Several active maintainers are not listed in MAINTAINERS.md.
CODEOWNERS has a duplicated /plugins/custom_calyptia/ entry.
Scope
CODEOWNERS:
Remove leonardo-albertovich from the global owners line.
Remove fujimotos, nokute78, pwhelan, koleini, pettitwesley and celalettin1286. Paths that had no other owner, including /src/stream_processor/, now fall back to the global owners. AWS
paths remain with the aws-fluent-bit-maintainers team. The Calyptia plugins are
owned by agup006 and alecholmes.
Group the Google plugins and add out_bigquery, out_chronicle and
the out_chronicle runtime test under JeffLuoo, next to out_stackdriver.
Add an Azure section: the out_azure* plugins and their tests are
owned by tanmaya-panda1.
Remove the /src/flb_lib.c entry so it falls back to the global
owners; niedbalski remains an owner of the CI, build and container
paths.
Drop the per-path entries that named edsiper (in_docker, out_datadog, out_es, .devcontainer). The global owners line
already covers them, so the three plugin paths fall back to it.
Remove the duplicated /plugins/custom_calyptia/ line.
MAINTAINERS.md:
Add Anurag Gupta, Patrick Stephens, Jose Lecaros, Alec Holmes
(Calyptia plugins), Shelby Hagman (Amazon plugins) and Tanmaya Panda
(Azure plugins).
Add an "Emeritus Maintainers" section for Leonardo Alminana, Fujimoto
Seiji, Takahiro Yamashita, Phillip Whelan, Masoud Koleini, Wesley
Pettit and Cedric Lamoriniere, with thanks for their contributions.
Refresh company names.
Testing
Documentation and metadata only, no code changes.
python .github/scripts/commit_prefix_check.py passes for the PR range.
Compatibility
No behavior change. Review requests for paths that lost their specific
owner now go to the global owners.
Summary by CodeRabbit
Chores
Updated repository ownership assignments across global, build, CI, plugin, and test areas, including changes to listed owners for Devcontainer and Calyptia Fleet paths.
Documentation
Updated the active maintainer roster, added six maintainers, and moved five former maintainers to the emeritus list. Updated Eduardo Silva’s company affiliation.
No end-user functionality changes are included in this update.
It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.
No actionable comments were generated in the recent review.
Walkthrough
The pull request updates the active and emeritus maintainer tables in MAINTAINERS.md. It also changes ownership assignments across repository, build, CI, plugin, and test paths in CODEOWNERS.
Changes: Maintainer and ownership updates
Layer / File(s)
Summary
Maintainer roster MAINTAINERS.md
The active table adds six maintainers and updates Eduardo Silva’s company. A new emeritus table lists seven people and their former component areas.
Repository, build, and CI ownership CODEOWNERS
Global, build, and CI ownership assignments change. The /src/flb_lib.c ownership rule is removed, and the /src/flb_signv4.c owners remain unchanged.
Plugin and test ownership CODEOWNERS
Plugin ownership rules are removed, retained, or added. Ownership rules are added for Azure and Chronicle tests. Devcontainer and Calyptia Fleet ownership changes.
This change updates the maintainer roster and code-owner assignments only, with no behavior change to the software. No actionable merge-blocking risk was identified.
Pre-merge checks: 4
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title "maintainers: update maintainers list and code owners" matches the main change. The PR updates MAINTAINERS.md (active maintainer table, new Emeritus section, company updates) and CODEOWNERS …
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Title check
Explanation
The title "maintainers: update maintainers list and code owners" matches the main change. The PR updates MAINTAINERS.md (active maintainer table, new Emeritus section, company updates) and CODEOWNERS (ownership rules for Google, Azure, and other paths). The title is specific, follows the component-prefix style, and does not describe unrelated work. It does not list every detail, which is expected.
Finishing Touches
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
File: No configuration file from this repository applied
Commit: Unavailable
Layers: defaults
Review profile: CHILL
Plan: Advanced
Run ID: 24204717-165f-4662-9935-f3e5f07949b3
Commits
Reviewing files that changed from the base of the PR and between a7014bc and f912c9c.
Files selected for processing: 1
CODEOWNERS
Files skipped from review as they are similar to previous changes: 1
CODEOWNERS
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 6 remain after this review.
Bring MAINTAINERS.md and CODEOWNERS in line with who maintains the
project today.
In MAINTAINERS.md, list the current maintainers with their areas,
adding the people who look after CI and packaging and the AWS, Azure
and Calyptia plugins. Add an Emeritus Maintainers section for former
maintainers and code owners, with thanks for their contributions,
and refresh company names.
In CODEOWNERS, remove owners who are no longer active or have no
write access to the repository; paths left without a specific owner
fall back to the global owners. Add owners for the Google, Azure and
Calyptia plugins and remove a duplicated entry.
Signed-off-by: Eduardo Silva <eduardo@chronosphere.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
MAINTAINERS.mdandCODEOWNERSno longer reflect who maintains theproject today:
the repository, so GitHub reports those lines as invalid.
MAINTAINERS.md.CODEOWNERShas a duplicated/plugins/custom_calyptia/entry.Scope
CODEOWNERS:leonardo-albertovichfrom the global owners line.fujimotos,nokute78,pwhelan,koleini,pettitwesleyandcelalettin1286. Paths that had no other owner, including/src/stream_processor/, now fall back to the global owners. AWSpaths remain with the
aws-fluent-bit-maintainersteam. The Calyptia plugins areowned by
agup006andalecholmes.out_bigquery,out_chronicleandthe
out_chronicleruntime test underJeffLuoo, next toout_stackdriver.out_azure*plugins and their tests areowned by
tanmaya-panda1./src/flb_lib.centry so it falls back to the globalowners;
niedbalskiremains an owner of the CI, build and containerpaths.
edsiper(in_docker,out_datadog,out_es,.devcontainer). The global owners linealready covers them, so the three plugin paths fall back to it.
/plugins/custom_calyptia/line.MAINTAINERS.md:(Calyptia plugins), Shelby Hagman (Amazon plugins) and Tanmaya Panda
(Azure plugins).
Seiji, Takahiro Yamashita, Phillip Whelan, Masoud Koleini, Wesley
Pettit and Cedric Lamoriniere, with thanks for their contributions.
Testing
Documentation and metadata only, no code changes.
python .github/scripts/commit_prefix_check.pypasses for the PR range.Compatibility
No behavior change. Review requests for paths that lost their specific
owner now go to the global owners.
Summary by CodeRabbit