Skip to content

CORENET-7238: Update OWNERS file - #145

Open
taanyas wants to merge 1 commit into
openshift:mainfrom
taanyas:CORENET-7238
Open

CORENET-7238: Update OWNERS file#145
taanyas wants to merge 1 commit into
openshift:mainfrom
taanyas:CORENET-7238

Conversation

@taanyas

@taanyas taanyas commented Jun 8, 2026

Copy link
Copy Markdown

Update OWNERS file - remove inactive and non-Red Hat members, add full corenetworking team as reviewers and updated approvers.

Removed: zshi-redhat, dougbtv, pliurh
Added reviewers: full corenetworking team
Added approvers: LionelJouin, danwinship, SchSeba, wizhaoredhat, bpickard22, fedepaol

Jira: https://issues.redhat.com/browse/CORENET-7238

Summary by CodeRabbit

  • Chores
    • Updated reviewer and approver assignments for the Networking/multus component.
    • Added the current designated reviewers and approvers and removed outdated assignments.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 8, 2026
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: cc7ac4f0-a4b4-4c1e-9430-74909cfafd64

📥 Commits

Reviewing files that changed from the base of the PR and between e0fc86d and 1e22584.

📒 Files selected for processing (1)
  • OWNERS

Walkthrough

The OWNERS file updates the Networking/multus reviewer roster to 26 users and the approver roster to six users. Previous reviewer and approver entries are removed.

Changes

OWNERS assignments

Layer / File(s) Summary
Reviewer and approver roster update
OWNERS
The reviewer roster now contains 26 specified users. The approver roster now contains fedepaol, SchSeba, bpickard22, LionelJouin, danwinship, and wizhaoredhat.

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

Suggested reviewers: schseba, bpickard22

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 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 main change: updating the OWNERS file.
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.
Stable And Deterministic Test Names ✅ Passed The pull request changes only OWNERS; the patch contains no Ginkgo test-title declarations or test files to assess.
Test Structure And Quality ✅ Passed Not applicable: the commit changes only OWNERS; it adds or modifies no Ginkgo test code, setup, cleanup, waits, or assertions.
Microshift Test Compatibility ✅ Passed The pull request changes only the OWNERS file. It adds no Ginkgo e2e tests or test references that require MicroShift compatibility review.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The pull request changes only the OWNERS file and adds no Ginkgo e2e tests or other test code requiring SNO compatibility review.
Topology-Aware Scheduling Compatibility ✅ Passed The pull request changes only the root OWNERS file; it adds no deployment manifests, operator code, controllers, or scheduling constraints.
Ote Binary Stdout Contract ✅ Passed The commit changes only the metadata file OWNERS; no process-level code or stdout-writing paths changed.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The patch changes only the root OWNERS file. It adds no Ginkgo tests, IP handling, URLs, image pulls, or external connectivity.
No-Weak-Crypto ✅ Passed HEAD changes only the metadata-only OWNERS file; the diff contains reviewer and approver names, with no weak-crypto or secret-handling terms.
Container-Privileges ✅ Passed The PR changes only OWNERS. No container or Kubernetes manifest changes, and no listed privilege indicators appear in the diff.
No-Sensitive-Data-In-Logs ✅ Passed The pull request changes only OWNERS reviewer and approver metadata. No logging code or sensitive-data fields appear in the diff or updated file.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@openshift-ci-robot

openshift-ci-robot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

@taanyas: This pull request references CORENET-7238 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set.

Details

In response to this:

Update OWNERS file - remove inactive members, add current team members.

Removed: zshi-redhat, dougbtv, pliurh
Added: LionelJouin, danwinship, taanyas, raphaelvrosa

Jira: https://issues.redhat.com/browse/CORENET-7238

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci
openshift-ci Bot requested review from SchSeba and bpickard22 June 8, 2026 07:11

@raphaelvrosa raphaelvrosa left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 8, 2026
@openshift-ci

openshift-ci Bot commented Jun 8, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: raphaelvrosa, taanyas
Once this PR has been reviewed and has the lgtm label, please assign dougbtv for approval. For more information see the Code Review Process.

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

@openshift-ci openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label Jul 17, 2026
@openshift-ci

openshift-ci Bot commented Jul 17, 2026

Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@openshift-ci openshift-ci Bot added the do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. label Jul 17, 2026
@SchSeba

SchSeba commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Hi @taanyas please check #145 (comment)

Signed-off-by: Tanya Singh <tasingh@redhat.com>
@openshift-ci openshift-ci Bot removed the do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. label Aug 7, 2026
@openshift-ci

openshift-ci Bot commented Aug 7, 2026

Copy link
Copy Markdown

@taanyas: all tests passed!

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants