CORENET-7234: Update OWNERS file

[raphaelvrosa]

Summary by CodeRabbit

• Chores
  • Updated the project's code review and approval configuration.

[openshift-ci-robot]

@raphaelvrosa: This pull request references CORENET-7234 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:

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.

[coderabbitai]

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: Enterprise

Run ID: 6dedd056-2092-4d0d-9372-191a93a584ea

📥 Commits

Reviewing files that changed from the base of the PR and between 12d99f3 and 845c255.

📒 Files selected for processing (1)

• OWNERS

---

Walkthrough

The OWNERS file was updated to modify project team membership. The reviewers list was revised by removing five previous members and adding four new members while keeping three existing reviewers. The approvers list was similarly updated to six members, reflecting leadership changes in the multus-admission-controller project.

Changes

OWNERS file update

Layer / File(s)	Summary
Reviewers and approvers update OWNERS	reviewers list updated by removing dougbtv, zshi, s1061123, aneeshkp, pliurh and adding taanyas, raphaelvrosa, SchSeba, danwinship, LionelJouin while retaining fepan, jeremyeder, bpickard22. approvers list narrowed to LionelJouin, danwinship, SchSeba, fepan, jeremyeder, bpickard22.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 14 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Test Structure And Quality	⚠️ Warning	Test code added in PR lacks meaningful assertion failure messages - all 11 Expect() assertions omit optional message parameters needed for diagnostics.	Add failure messages to all Expect() calls: e.g., Expect(err).NotTo(HaveOccurred(), "failed to create pod") to help diagnose test failures.

✅ Passed checks (14 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: updating the OWNERS file with new reviewer and approver entries. 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.
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	All Ginkgo test names use stable, deterministic static strings. No dynamic content like timestamps, UUIDs, pod names, or node names found in test titles.
Microshift Test Compatibility	✅ Passed	The PR adds Ginkgo unit tests (not e2e tests) that only use standard Kubernetes APIs. The MicroShift compatibility check applies to e2e tests; these are unit tests.
Single Node Openshift (Sno) Test Compatibility	✅ Passed	PR only updates OWNERS file; no new Ginkgo e2e tests are added, so SNO compatibility check is not applicable.
Topology-Aware Scheduling Compatibility	✅ Passed	This PR only updates the OWNERS file with new reviewers/approvers; it does not modify deployment manifests, operator code, or controllers, so the topology-aware scheduling check is not applicable.
Ote Binary Stdout Contract	✅ Passed	PR only updates OWNERS and config files; no code changes that could write to stdout at process level, making OTE Binary Stdout Contract check inapplicable.
Ipv6 And Disconnected Network Test Compatibility	✅ Passed	This PR only modifies the OWNERS file to update reviewer and approver lists. No new Ginkgo e2e tests were added, so the IPv6/disconnected network check does not apply.
No-Weak-Crypto	✅ Passed	PR only modifies OWNERS file (configuration); no code changes or crypto-related implementations are present or modified.
Container-Privileges	✅ Passed	PR only updates OWNERS file; no container/K8s manifest changes. Existing deployment manifests lack privileged settings.
No-Sensitive-Data-In-Logs	✅ Passed	The PR only modifies the OWNERS file (metadata with GitHub usernames). Existing logging code contains no sensitive data exposure patterns and the PR makes no changes to logging statements.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[bpickard22]

/lgtm

[bpickard22]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bpickard22, raphaelvrosa

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [bpickard22]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bpickard22, raphaelvrosa

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [bpickard22]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-ci]

@raphaelvrosa: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/security	845c255	link	false	/test security

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.