Skip to content

OWNERS: Auto Sync OWNERS files from community membership#427

Open
ti-chi-bot wants to merge 2 commits into8.10.tikvfrom
bot/update-owners-1769753216798
Open

OWNERS: Auto Sync OWNERS files from community membership#427
ti-chi-bot wants to merge 2 commits into8.10.tikvfrom
bot/update-owners-1769753216798

Conversation

@ti-chi-bot
Copy link
Member

@ti-chi-bot ti-chi-bot commented Jan 30, 2026

Check List

Tests

  • No need to test
    • I checked and no code files have been changed.

Summary by CodeRabbit

  • Chores
    • Simplified repository governance configuration by centralizing approver and reviewer group assignments
    • Enhanced maintainability of access control through a dedicated configuration mapping

✏️ Tip: You can customize this high-level summary in your review settings.

[skip ci] Update OWNERS file

skip-checks: true
Signed-off-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
[skip ci] Update OWNERS file

skip-checks: true
Signed-off-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot bot added the dco-signoff: yes Indicates the PR's author has signed the dco. label Jan 30, 2026
@ti-chi-bot
Copy link

ti-chi-bot bot commented Jan 30, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign yiwu-arbug for approval. For more information see the Code Review Process.
Please ensure that each of them provides their approval before proceeding.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 30, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

The changes consolidate approver and reviewer assignments by replacing explicit user lists in the OWNERS file with a filters-based approach that references centralized governance groups. A new OWNERS_ALIASES file defines these groups and their memberships, centralizing identity management.

Changes

Cohort / File(s) Summary
Governance Configuration
OWNERS, OWNERS_ALIASES
Restructures ownership and review approvals from explicit user lists to centralized group references. OWNERS now uses filters mapping to sig-community-approvers and sig-community-reviewers groups defined in the new OWNERS_ALIASES file.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 The lists grow long, but now we see,
A central home where groups shall be,
With aliases clear, all neat and tight,
Our governance code shines ever bright! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: syncing OWNERS files from community membership definitions, which aligns with the actual modifications to OWNERS and OWNERS_ALIASES files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bot/update-owners-1769753216798

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.

@ti-chi-bot
Copy link
Member Author

/release-note-none

@ti-chi-bot ti-chi-bot added approved lgtm skip-issue-check Indicates that a PR no need to check linked issue. labels Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved dco-signoff: yes Indicates the PR's author has signed the dco. lgtm size/L Denotes a PR that changes 100-499 lines, ignoring generated files. skip-issue-check Indicates that a PR no need to check linked issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant