Skip to content

UPSTREAM: <carry>: Add OWNERS file#15

Merged
kaovilai merged 1 commit intooadp-devfrom
add-owners-oadp-dev
Apr 1, 2026
Merged

UPSTREAM: <carry>: Add OWNERS file#15
kaovilai merged 1 commit intooadp-devfrom
add-owners-oadp-dev

Conversation

@mpryc
Copy link
Copy Markdown

@mpryc mpryc commented Apr 1, 2026

Summary

  • Add OWNERS file matching oadp-cli project conventions

Signed-off-by: Michal Pryc mpryc@redhat.com

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated repository governance configuration to define approval and review roles.

Note: This change is internal repository maintenance with no impact on end-user functionality.

Signed-off-by: Michal Pryc <mpryc@redhat.com>
Co-authored-by: Michal Pryc <mpryc@redhat.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@oadp-snyk
Copy link
Copy Markdown

oadp-snyk commented Apr 1, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 994dc62f-11c7-4f58-a8e9-d95c5e2565ca

📥 Commits

Reviewing files that changed from the base of the PR and between 30d2789 and 22e54a7.

📒 Files selected for processing (1)
  • OWNERS

📝 Walkthrough

Walkthrough

A new OWNERS file is introduced to define repository governance, listing GitHub usernames for the approvers and reviewers roles. The approvers list includes eight team members, while reviewers includes the same set minus two members.

Changes

Cohort / File(s) Summary
Repository Governance
OWNERS
Added new governance configuration file defining approvers and reviewers role lists with GitHub usernames for repository access control.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

  • OWNERS file oadp-cli#152: This PR directly addresses the need to establish OWNERS governance configuration, completing the same initiative to define approvers and reviewers for repository management.

Poem

🐰 A kingdom needs keepers, they say,
So we've listed the guardians today!
Approvers and reviewers aligned,
Eight noble souls, carefully signed. 🏛️

🚥 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 accurately describes the main change: adding an OWNERS file to the repository. It is concise, clear, and directly related to the changeset.
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
  • Commit unit tests in branch add-owners-oadp-dev

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.

Copy link
Copy Markdown
Member

@kaovilai kaovilai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Apr 1, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kaovilai, mpryc

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

@kaovilai kaovilai merged commit 7e7ed77 into oadp-dev Apr 1, 2026
10 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants