Skip to content

ci: make the DevOps team code owner of the CI pipeline - #1069

Merged
orbalayla-nvidia merged 1 commit into
NVIDIA:mainfrom
orbalayla-nvidia:ci/codeowners-devops-team
Sep 30, 2026
Merged

orbalayla-nvidia merged 1 commit into
NVIDIA:mainfrom
orbalayla-nvidia:ci/codeowners-devops-team

Conversation

@orbalayla-nvidia

@orbalayla-nvidia orbalayla-nvidia commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Make @NVIDIA/nbu-swx-cloudai-devops the code owner of .ci/ and .github/workflows/blossom-ci.yml, so changes to the CI pipeline and the workflow that triggers it are auto-requested from DevOps.
  • The team is synced from the Access-Github-nbu-swx-cloudai-devops DL and already has Write on the repository, which GitHub requires for a code-owner team.
  • The repository-wide * owners are unchanged. The new lines come after it, since the last matching pattern wins.

Test Plan

  • pre-commit run --files .github/CODEOWNERS passes.

  • GitHub's CODEOWNERS validator on this PR's head reports no errors, so the team resolves and has the access it needs:

    $ gh api "repos/NVIDIA/cloudai/codeowners/errors?ref=refs/pull/1069/head"
    {"errors":[]}
    

Additional Notes

CODEOWNERS only requests the review. "Require review from Code Owners" stays off in the main ruleset, because with the * line it would make one of the four listed owners mandatory on every pull request. Approval for these paths will be enforced separately with a path-scoped Required reviewers rule in the ruleset.

The .ci/ pipeline and the Blossom workflow that triggers it had no owner
beyond the repository-wide list, so a change to either was only ever
auto-requested from reviewers outside DevOps. Assign both to
@NVIDIA/nbu-swx-cloudai-devops, synced from the
Access-Github-nbu-swx-cloudai-devops DL. The repository-wide owners
stay unchanged for everything else.

Refs: HPCINFRA-4886

Signed-off-by: Or Balayla <obalayla@nvidia.com>
@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: NVIDIA/cloudai/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: 925156f0-2b89-4ccb-8fa7-3753f3d04816

📥 Commits

Reviewing files that changed from the base of the PR and between 559e9b5 and 8927eb9.

📒 Files selected for processing (1)
  • .github/CODEOWNERS

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.


📝 Walkthrough

Walkthrough

The CODEOWNERS file adds ownership rules for .ci/ and .github/workflows/blossom-ci.yml.

Changes

CI Path Ownership

Layer / File(s) Summary
CI ownership rules
.github/CODEOWNERS
Adds rules that assign .ci/ and .github/workflows/blossom-ci.yml to the same owner team.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 8927e

This change only adds code owners for the CI paths, so it has no runtime impact. It requests reviews from the DevOps team on those paths. Whether approval is enforced depends on a repository ruleset outside this PR, which the author should confirm.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: assigning the DevOps team as code owner for the CI pipeline.
Description check ✅ Passed The description directly explains the CODEOWNERS changes, access requirements, validation results, and review-enforcement behavior.
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
🧪 Generate unit tests (beta)
  • Create a new PR

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

@orbalayla-nvidia
orbalayla-nvidia merged commit cf250dc into NVIDIA:main Sep 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants