Skip to content

OWNERS: sync master to release-7.5 - #70404

Closed
ti-chi-bot wants to merge 0 commit into
release-7.5from
bot/sync-owners-to-release-7.5
Closed

ti-chi-bot wants to merge 0 commit into
release-7.5from
bot/sync-owners-to-release-7.5

Conversation

@ti-chi-bot

@ti-chi-bot ti-chi-bot commented Aug 10, 2026

Copy link
Copy Markdown
Member

What problem does this PR solve?

Sync OWNERS* files from master to release-7.5.

What changed and how does it work?

This PR was auto-generated by Prow and only updates OWNERS and OWNERS_ALIASES files.

/release-note-none
skip-issue-check
lgtm
approved
cherry-pick-approved

Summary by CodeRabbit

  • Chores
    • Standardized contribution approval rules across project areas using path-specific ownership patterns.
    • Added consistent handling for build configuration files and specialized approval groups.
    • Updated ownership aliases, membership mappings, and related documentation links.
    • Added ownership coverage for several components and subpackages.
    • Removed obsolete ownership configurations and retired an outdated approval alias.

@ti-chi-bot ti-chi-bot added approved cherry-pick-approved Cherry pick PR approved by release team. lgtm skip-issue-check Indicates that a PR no need to check linked issue. labels Aug 10, 2026
@ti-chi-bot

ti-chi-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@ti-chi-bot ti-chi-bot Bot added do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. component/dumpling This is related to Dumpling of TiDB. sig/planner SIG: Planner size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed approved labels Aug 10, 2026
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1f68a369-207a-404e-b085-7e28eb1a8221

📥 Commits

Reviewing files that changed from the base of the PR and between 416a6fb and e84d71e.

📒 Files selected for processing (50)
  • OWNERS
  • OWNERS_ALIASES
  • br/OWNERS
  • br/cmd/tidb-lightning-ctl/OWNERS
  • br/cmd/tidb-lightning/OWNERS
  • br/pkg/lightning/OWNERS
  • dumpling/OWNERS
  • lightning/OWNERS
  • lightning/cmd/tidb-lightning-ctl/OWNERS
  • lightning/cmd/tidb-lightning/OWNERS
  • pkg/autoid_service/OWNERS
  • pkg/bindinfo/OWNERS
  • pkg/config/OWNERS
  • pkg/ddl/OWNERS
  • pkg/distsql/OWNERS
  • pkg/disttask/OWNERS
  • pkg/domain/OWNERS
  • pkg/dumpformat/OWNERS
  • pkg/dxf/OWNERS
  • pkg/executor/OWNERS
  • pkg/executor/aggfuncs/OWNERS
  • pkg/executor/aggregate/OWNERS
  • pkg/executor/importer/OWNERS
  • pkg/executor/join/OWNERS
  • pkg/executor/mppcoordmanager/OWNERS
  • pkg/executor/sortexec/OWNERS
  • pkg/executor/test/analyzetest/OWNERS
  • pkg/executor/unionexec/OWNERS
  • pkg/expression/OWNERS
  • pkg/infoschema/OWNERS
  • pkg/ingestor/OWNERS
  • pkg/lightning/OWNERS
  • pkg/lightning/config/OWNERS
  • pkg/lock/OWNERS
  • pkg/meta/OWNERS
  • pkg/meta/metadef/OWNERS
  • pkg/metrics/OWNERS
  • pkg/objstore/OWNERS
  • pkg/owner/OWNERS
  • pkg/parser/OWNERS
  • pkg/planner/OWNERS
  • pkg/resourcemanager/OWNERS
  • pkg/session/OWNERS
  • pkg/sessionctx/vardef/OWNERS
  • pkg/sessionctx/variable/OWNERS
  • pkg/statistics/OWNERS
  • pkg/table/OWNERS
  • pkg/tablecodec/OWNERS
  • pkg/tidb-binlog/OWNERS
  • pkg/util/naming/OWNERS
💤 Files with no reviewable changes (5)
  • br/cmd/tidb-lightning-ctl/OWNERS
  • br/pkg/lightning/OWNERS
  • br/cmd/tidb-lightning/OWNERS
  • pkg/tidb-binlog/OWNERS
  • pkg/disttask/OWNERS

📝 Walkthrough

Walkthrough

The PR replaces static OWNERS approver and reviewer lists with path-based rules. It adds synchronized SIG Community aliases and applies specific approval groups to BUILD.bazel, go.mod, and package-specific files.

Changes

Ownership pattern migration

Layer / File(s) Summary
Ownership aliases and repository defaults
OWNERS, OWNERS_ALIASES
Repository ownership rules now use filters. Alias memberships and documentation links were updated. Community approver and reviewer aliases were added.
Existing package filter migration
br/OWNERS, dumpling/OWNERS, pkg/*/OWNERS
Existing OWNERS files now route BUILD.bazel files to sig-community-approvers and retain package-specific fallback approvers and labels.
New package ownership boundaries
lightning/**/OWNERS, pkg/executor/**/OWNERS, pkg/sessionctx/**/OWNERS, pkg/*/OWNERS
New OWNERS files define local approval filters and disable parent-owner inheritance where specified. Several obsolete ownership files were removed.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Possibly related issues

  • pingcap/tidb#70390 — The PR adds BUILD.bazel filters with sig-community-approvers across nested OWNERS files.

Possibly related PRs

  • pingcap/tidb#70391 — Both PRs update nested OWNERS files with path-based BUILD.bazel approval rules.

Suggested labels: component/build

Poem

BUILD files hop through the tree,
Community approvers now agree.
SIG aliases gather in tune,
OWNERS rules change beneath the moon.
go.mod finds its careful guide,
A rabbit reviews with ears open wide.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the synchronization of OWNERS files from master to release-7.5.
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.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bot/sync-owners-to-release-7.5

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.

@ti-chi-bot

ti-chi-bot Bot commented Aug 13, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

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 size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick-approved Cherry pick PR approved by release team. component/dumpling This is related to Dumpling of TiDB. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. lgtm sig/planner SIG: Planner size/XS Denotes a PR that changes 0-9 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