Skip to content

UPSTREAM: <carry> Add OWNERS file#154

Merged
Joeavaikath merged 1 commit intomigtools:oadp-devfrom
mpryc:owners_ci_file
Mar 12, 2026
Merged

UPSTREAM: <carry> Add OWNERS file#154
Joeavaikath merged 1 commit intomigtools:oadp-devfrom
mpryc:owners_ci_file

Conversation

@mpryc
Copy link
Copy Markdown
Contributor

@mpryc mpryc commented Mar 12, 2026

Adds OWNERS file.

Why the changes were made

Required to enable prow CI.

Summary by CodeRabbit

  • Chores
    • Updated code review governance configuration to define authorization roles for managing changes.

Adds OWNERS file.

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 12, 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: be75e9d2-3f33-4575-8026-f5275e42ee83

📥 Commits

Reviewing files that changed from the base of the PR and between a6377ad and 9e81377.

📒 Files selected for processing (1)
  • OWNERS

📝 Walkthrough

Walkthrough

This pull request adds a new OWNERS file at the repository root containing two lists: approvers (8 users) and reviewers (6 users). These entries define authorization for code review and approval workflows.

Changes

Cohort / File(s) Summary
OWNERS Configuration
OWNERS
Added approvers list with 8 usernames and reviewers list with 6 usernames to define authorization for code review automation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

  • OWNERS file #152 — This PR mirrors the same OWNERS file structure and authorization entries as an identical change in the referenced repository.

Poem

🐰 A document so small, yet mighty and grand,
With names carefully listed across the land,
Approvers and reviewers standing in line,
To guide the code and make it shine!
The OWNERS file hops into place,
Bringing order to this coding space! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description includes a brief explanation of why the change was made (to enable prow CI), but lacks details on how to test the changes as required by the template. Add a 'How to test the changes made' section explaining how to verify the OWNERS file is correctly configured and enables prow CI as intended.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding an OWNERS file to the repository.
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

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.

@Joeavaikath Joeavaikath merged commit 111acba into migtools:oadp-dev Mar 12, 2026
16 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