chore: sync maintainer metadata with the organisation - #6
Merged
Andrei Kvapil (kvaps) merged 1 commit intoAug 14, 2026
Merged
Conversation
Two drifts between this repository's metadata and the organisation it describes. CODEOWNERS named @gecube and @nbykov0, who are no longer maintainers, and omitted @IvanHunters, @lexfrei and @myasnikovdaniil, who are. A CODEOWNERS entry is only honoured for principals with write access, and this repository had no team attached, so most of the handles resolved to the organisation's base read permission and were silently ignored — the file looked like ownership while providing none. The maintainers team has since been granted push access, so the list now actually routes. maintainers.yaml declared the team as project-maintainers. No such team exists; the organisation's team is maintainers, and this file exists to mirror the roster for drift detection, so it has to name the team that is really there. Assisted-By: Claude Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe changes update CNCF maintainer metadata. ChangesMaintainer metadata
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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. Comment |
Andrei Kvapil (kvaps)
marked this pull request as ready for review
August 12, 2026 20:58
Aleksei Sviridkin (lexfrei)
approved these changes
Aug 12, 2026
myasnikovdaniil
approved these changes
Aug 13, 2026
mattia-eleuteri
approved these changes
Aug 13, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What this PR does
Two drifts between this repository's metadata and the organisation it describes.
CODEOWNERSnamed George Gaál (@gecube) and Nikita (@nbykov0), who are no longer maintainers, and omitted IvanHunters, Aleksei Sviridkin (@lexfrei) and myasnikovdaniil, who are. It is now the current roster frommaintainers.yaml.A CODEOWNERS entry is only honoured for principals with write access, and this repository had no team attached — so five of the eight handles resolved to the organisation's base
readpermission and were silently ignored, and one (Nikita (@nbykov0)) was not a collaborator at all. Only three of the eight actually routed. Themaintainersteam has since been grantedpushhere, so the list now does what it looks like it does.maintainers.yamldeclared the team asproject-maintainers. No such team exists in the organisation; the team ismaintainers. Since this file exists to mirror the roster for drift detection, it has to name the team that is really there.The roster itself (9 maintainers, 3 reviewers) was already correct and is unchanged.
Note that
cncf/foundation#1489, which updatesproject-maintainers.csvon the CNCF side, is still open — this PR does not close that loop.Summary by CodeRabbit