Skip to content

Conversation

jcaamano
Copy link
Contributor

@jcaamano jcaamano commented Aug 6, 2025

Removing trozet as he left the company. Adding tssurya as team lead replacement.

Adding pliurh as approver replacing dougbtv who will be eventually removed after some soak period as he moved on to other unrelated projects. pliurh has demonstrated extensive experience working on this repository througout his work on sdn->ovnk different migration modes.

Added bpickard22 to help as a reviewer for multus related changes as a replacement for dougbtv.

Adding the overall core network team as responsible to perform reviews.

(cherry picked from commit 1704438) (cherry picked from commit d90db96)

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Aug 6, 2025
@openshift-ci-robot
Copy link
Contributor

@jcaamano: This pull request explicitly references no jira issue.

In response to this:

Removing trozet as he left the company. Adding tssurya as team lead replacement.

Adding pliurh as approver replacing dougbtv who will be eventually removed after some soak period as he moved on to other unrelated projects. pliurh has demonstrated extensive experience working on this repository througout his work on sdn->ovnk different migration modes.

Added bpickard22 to help as a reviewer for multus related changes as a replacement for dougbtv.

Adding the overall core network team as responsible to perform reviews.

(cherry picked from commit 1704438) (cherry picked from commit d90db96)

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.

@jcaamano
Copy link
Contributor Author

jcaamano commented Aug 6, 2025

/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-aws-ovn-upgrade
/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-aws-ovn-upgrade-ipsec
/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-azure-ovn-upgrade
/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-gcp-ovn-upgrade
/override ci/prow/4.19-upgrade-from-stable-4.18-images
/override ci/prow/e2e-aws-hypershift-ovn-kubevirt
/override ci/prow/e2e-aws-ovn-hypershift-conformance
/override ci/prow/e2e-aws-ovn-local-to-shared-gateway-mode-migration
/override ci/prow/e2e-aws-ovn-serial
/override ci/prow/e2e-aws-ovn-serial-ipsec
/override ci/prow/e2e-aws-ovn-shared-to-local-gateway-mode-migration
/override ci/prow/e2e-aws-ovn-single-node
/override ci/prow/e2e-aws-ovn-upgrade
/override ci/prow/e2e-aws-ovn-upgrade-ipsec
/override ci/prow/e2e-aws-ovn-windows
/override ci/prow/e2e-azure-ovn
/override ci/prow/e2e-azure-ovn-upgrade
/override ci/prow/e2e-gcp-ovn
/override ci/prow/e2e-gcp-ovn-upgrade
/override ci/prow/e2e-metal-ipi-ovn-dualstack-bgp-local-gw-techpreview
/override ci/prow/e2e-metal-ipi-ovn-dualstack-bgp-techpreview
/override ci/prow/e2e-metal-ipi-ovn-ipv6
/override ci/prow/e2e-metal-ipi-ovn-ipv6-ipsec
/override ci/prow/e2e-network-mtu-migration-ovn-ipv4
/override ci/prow/e2e-network-mtu-migration-ovn-ipv6
/override ci/prow/e2e-openstack-ovn
/override ci/prow/e2e-ovn-hybrid-step-registry
/override ci/prow/e2e-ovn-ipsec-step-registry
/override ci/prow/e2e-ovn-step-registry
/override ci/prow/e2e-vsphere-ovn
/override ci/prow/e2e-vsphere-ovn-dualstack
/override ci/prow/e2e-vsphere-ovn-dualstack-primaryv6
/override ci/prow/hypershift-e2e-aks

@openshift-ci openshift-ci bot requested review from danwinship and dougbtv August 6, 2025 14:10
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 6, 2025
Copy link
Contributor

openshift-ci bot commented Aug 6, 2025

@jcaamano: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • ci/prow/4.19-upgrade-from-stable-4.18-e2e-aws-ovn-upgrade
  • ci/prow/4.19-upgrade-from-stable-4.18-e2e-aws-ovn-upgrade-ipsec
  • ci/prow/4.19-upgrade-from-stable-4.18-e2e-azure-ovn-upgrade
  • ci/prow/4.19-upgrade-from-stable-4.18-e2e-gcp-ovn-upgrade
  • ci/prow/4.19-upgrade-from-stable-4.18-images
  • ci/prow/e2e-aws-ovn-serial-ipsec
  • ci/prow/e2e-metal-ipi-ovn-dualstack-bgp-local-gw-techpreview
  • ci/prow/e2e-metal-ipi-ovn-dualstack-bgp-techpreview
  • ci/prow/hypershift-e2e-aks

Only the following failed contexts/checkruns were expected:

  • ci/prow/4.16-upgrade-from-stable-4.15-e2e-aws-ovn-upgrade
  • ci/prow/4.16-upgrade-from-stable-4.15-e2e-aws-ovn-upgrade-ipsec
  • ci/prow/4.16-upgrade-from-stable-4.15-e2e-azure-ovn-upgrade
  • ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
  • ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-upgrade
  • ci/prow/4.16-upgrade-from-stable-4.15-images
  • ci/prow/e2e-aws-hypershift-ovn-kubevirt
  • ci/prow/e2e-aws-live-migration-sdn-ovn
  • ci/prow/e2e-aws-live-migration-sdn-ovn-rollback
  • ci/prow/e2e-aws-ovn-hypershift-conformance
  • ci/prow/e2e-aws-ovn-local-to-shared-gateway-mode-migration
  • ci/prow/e2e-aws-ovn-network-migration
  • ci/prow/e2e-aws-ovn-serial
  • ci/prow/e2e-aws-ovn-shared-to-local-gateway-mode-migration
  • ci/prow/e2e-aws-ovn-single-node
  • ci/prow/e2e-aws-ovn-upgrade
  • ci/prow/e2e-aws-ovn-upgrade-ipsec
  • ci/prow/e2e-aws-ovn-windows
  • ci/prow/e2e-aws-sdn-upgrade
  • ci/prow/e2e-azure-ovn
  • ci/prow/e2e-azure-ovn-upgrade
  • ci/prow/e2e-gcp-ovn
  • ci/prow/e2e-gcp-ovn-upgrade
  • ci/prow/e2e-metal-ipi-ovn-ipv6
  • ci/prow/e2e-metal-ipi-ovn-ipv6-ipsec
  • ci/prow/e2e-network-mtu-migration-ovn-ipv4
  • ci/prow/e2e-network-mtu-migration-ovn-ipv6
  • ci/prow/e2e-openstack-ovn
  • ci/prow/e2e-ovn-hybrid-step-registry
  • ci/prow/e2e-ovn-ipsec-step-registry
  • ci/prow/e2e-ovn-step-registry
  • ci/prow/e2e-vsphere-ovn
  • ci/prow/e2e-vsphere-ovn-dualstack
  • ci/prow/e2e-vsphere-ovn-dualstack-primaryv6
  • ci/prow/images
  • ci/prow/lint
  • ci/prow/okd-scos-images
  • ci/prow/security
  • ci/prow/unit
  • ci/prow/verify
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-hypershift-ovn-kubevirt
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-hypershift-conformance
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-local-to-shared-gateway-mode-migration
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-serial
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-shared-to-local-gateway-mode-migration
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-single-node
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-upgrade
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-upgrade-ipsec
  • pull-ci-openshift-cluster-network-operator-master-e2e-aws-ovn-windows
  • pull-ci-openshift-cluster-network-operator-master-e2e-azure-ovn
  • pull-ci-openshift-cluster-network-operator-master-e2e-azure-ovn-upgrade
  • pull-ci-openshift-cluster-network-operator-master-e2e-gcp-ovn
  • pull-ci-openshift-cluster-network-operator-master-e2e-gcp-ovn-upgrade
  • pull-ci-openshift-cluster-network-operator-master-e2e-metal-ipi-ovn-ipv6
  • pull-ci-openshift-cluster-network-operator-master-e2e-metal-ipi-ovn-ipv6-ipsec
  • pull-ci-openshift-cluster-network-operator-master-e2e-network-mtu-migration-ovn-ipv4
  • pull-ci-openshift-cluster-network-operator-master-e2e-network-mtu-migration-ovn-ipv6
  • pull-ci-openshift-cluster-network-operator-master-e2e-openstack-ovn
  • pull-ci-openshift-cluster-network-operator-master-e2e-ovn-hybrid-step-registry
  • pull-ci-openshift-cluster-network-operator-master-e2e-ovn-ipsec-step-registry
  • pull-ci-openshift-cluster-network-operator-master-e2e-ovn-step-registry
  • pull-ci-openshift-cluster-network-operator-master-e2e-vsphere-ovn
  • pull-ci-openshift-cluster-network-operator-master-e2e-vsphere-ovn-dualstack
  • pull-ci-openshift-cluster-network-operator-master-e2e-vsphere-ovn-dualstack-primaryv6
  • pull-ci-openshift-cluster-network-operator-master-images
  • pull-ci-openshift-cluster-network-operator-master-lint
  • pull-ci-openshift-cluster-network-operator-master-okd-scos-images
  • pull-ci-openshift-cluster-network-operator-master-security
  • pull-ci-openshift-cluster-network-operator-master-unit
  • pull-ci-openshift-cluster-network-operator-master-verify
  • pull-ci-openshift-cluster-network-operator-release-4.12-e2e-aws-ovn-network-migration
  • pull-ci-openshift-cluster-network-operator-release-4.12-e2e-aws-sdn-upgrade
  • pull-ci-openshift-cluster-network-operator-release-4.15-e2e-aws-live-migration-sdn-ovn
  • pull-ci-openshift-cluster-network-operator-release-4.15-e2e-aws-live-migration-sdn-ovn-rollback
  • pull-ci-openshift-cluster-network-operator-release-4.16-4.16-upgrade-from-stable-4.15-e2e-aws-ovn-upgrade
  • pull-ci-openshift-cluster-network-operator-release-4.16-4.16-upgrade-from-stable-4.15-e2e-aws-ovn-upgrade-ipsec
  • pull-ci-openshift-cluster-network-operator-release-4.16-4.16-upgrade-from-stable-4.15-e2e-azure-ovn-upgrade
  • pull-ci-openshift-cluster-network-operator-release-4.16-4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
  • pull-ci-openshift-cluster-network-operator-release-4.16-4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-upgrade
  • pull-ci-openshift-cluster-network-operator-release-4.16-4.16-upgrade-from-stable-4.15-images
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-aws-ovn-upgrade
/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-aws-ovn-upgrade-ipsec
/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-azure-ovn-upgrade
/override ci/prow/4.19-upgrade-from-stable-4.18-e2e-gcp-ovn-upgrade
/override ci/prow/4.19-upgrade-from-stable-4.18-images
/override ci/prow/e2e-aws-hypershift-ovn-kubevirt
/override ci/prow/e2e-aws-ovn-hypershift-conformance
/override ci/prow/e2e-aws-ovn-local-to-shared-gateway-mode-migration
/override ci/prow/e2e-aws-ovn-serial
/override ci/prow/e2e-aws-ovn-serial-ipsec
/override ci/prow/e2e-aws-ovn-shared-to-local-gateway-mode-migration
/override ci/prow/e2e-aws-ovn-single-node
/override ci/prow/e2e-aws-ovn-upgrade
/override ci/prow/e2e-aws-ovn-upgrade-ipsec
/override ci/prow/e2e-aws-ovn-windows
/override ci/prow/e2e-azure-ovn
/override ci/prow/e2e-azure-ovn-upgrade
/override ci/prow/e2e-gcp-ovn
/override ci/prow/e2e-gcp-ovn-upgrade
/override ci/prow/e2e-metal-ipi-ovn-dualstack-bgp-local-gw-techpreview
/override ci/prow/e2e-metal-ipi-ovn-dualstack-bgp-techpreview
/override ci/prow/e2e-metal-ipi-ovn-ipv6
/override ci/prow/e2e-metal-ipi-ovn-ipv6-ipsec
/override ci/prow/e2e-network-mtu-migration-ovn-ipv4
/override ci/prow/e2e-network-mtu-migration-ovn-ipv6
/override ci/prow/e2e-openstack-ovn
/override ci/prow/e2e-ovn-hybrid-step-registry
/override ci/prow/e2e-ovn-ipsec-step-registry
/override ci/prow/e2e-ovn-step-registry
/override ci/prow/e2e-vsphere-ovn
/override ci/prow/e2e-vsphere-ovn-dualstack
/override ci/prow/e2e-vsphere-ovn-dualstack-primaryv6
/override ci/prow/hypershift-e2e-aks

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.

@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 Aug 6, 2025
@jcaamano
Copy link
Contributor Author

jcaamano commented Aug 7, 2025

/override ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-aws-ovn-upgrade
ci/prow/e2e-gcp-ovn
ci/prow/e2e-gcp-ovn-upgrade

@jcaamano jcaamano changed the title [release-4.17]: NO-JIRA: Update CNO reviewers/approvers [release-4.16]: NO-JIRA: Update CNO reviewers/approvers Aug 7, 2025
Copy link
Contributor

openshift-ci bot commented Aug 7, 2025

@jcaamano: Overrode contexts on behalf of jcaamano: ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade

In response to this:

/override ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-aws-ovn-upgrade
ci/prow/e2e-gcp-ovn
ci/prow/e2e-gcp-ovn-upgrade

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.

@jcaamano
Copy link
Contributor Author

jcaamano commented Aug 7, 2025

/override ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
/override ci/prow/e2e-aws-ovn-upgrade
/override ci/prow/e2e-gcp-ovn
/override ci/prow/e2e-gcp-ovn-upgrade

Copy link
Contributor

openshift-ci bot commented Aug 7, 2025

@jcaamano: Overrode contexts on behalf of jcaamano: ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade, ci/prow/e2e-aws-ovn-upgrade, ci/prow/e2e-gcp-ovn, ci/prow/e2e-gcp-ovn-upgrade

In response to this:

/override ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
/override ci/prow/e2e-aws-ovn-upgrade
/override ci/prow/e2e-gcp-ovn
/override ci/prow/e2e-gcp-ovn-upgrade

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.

@kyrtapz
Copy link
Contributor

kyrtapz commented Aug 7, 2025

/lgtm
/label backport-risk-assessed

@openshift-ci openshift-ci bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Aug 7, 2025
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Aug 7, 2025
Removing trozet as he left the company. Adding tssurya as team lead
replacement.

Adding pliurh as approver replacing dougbtv who will be eventually
removed after some soak period as he moved on to other unrelated
projects. pliurh has demonstrated extensive experience working on this
repository througout his work on sdn->ovnk different migration modes.

Added bpickard22 to help as a reviewer for multus related changes as a
replacement for dougbtv.

Adding the overall core network team as responsible to perform reviews.

Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
(cherry picked from commit 1704438)
(cherry picked from commit d90db96)
@jcaamano jcaamano force-pushed the approvers-master-r16 branch from 6fc50b3 to f3d55af Compare August 11, 2025 15:22
@openshift-ci openshift-ci bot removed lgtm Indicates that a PR is ready to be merged. do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. labels Aug 11, 2025
@kyrtapz
Copy link
Contributor

kyrtapz commented Aug 11, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Aug 11, 2025
Copy link
Contributor

openshift-ci bot commented Aug 11, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jcaamano, kyrtapz

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

The pull request process is described here

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

@asood-rh
Copy link

/label cherry-pick-approved

@openshift-ci openshift-ci bot added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Aug 11, 2025
Copy link
Contributor

openshift-ci bot commented Aug 11, 2025

@jcaamano: The following tests 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/e2e-vsphere-ovn-dualstack-primaryv6 6fc50b3 link false /test e2e-vsphere-ovn-dualstack-primaryv6
ci/prow/e2e-aws-hypershift-ovn-kubevirt f3d55af link false /test e2e-aws-hypershift-ovn-kubevirt
ci/prow/e2e-aws-ovn-windows f3d55af link true /test e2e-aws-ovn-windows
ci/prow/security f3d55af link false /test security
ci/prow/e2e-gcp-ovn-upgrade f3d55af link true /test e2e-gcp-ovn-upgrade
ci/prow/e2e-metal-ipi-ovn-ipv6-ipsec f3d55af link false /test e2e-metal-ipi-ovn-ipv6-ipsec
ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-upgrade f3d55af link false /test 4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-upgrade
ci/prow/e2e-aws-ovn-network-migration f3d55af link true /test e2e-aws-ovn-network-migration
ci/prow/e2e-azure-ovn-upgrade f3d55af link true /test e2e-azure-ovn-upgrade
ci/prow/4.16-upgrade-from-stable-4.15-e2e-aws-ovn-upgrade-ipsec f3d55af link false /test 4.16-upgrade-from-stable-4.15-e2e-aws-ovn-upgrade-ipsec
ci/prow/e2e-aws-ovn-upgrade-ipsec f3d55af link true /test e2e-aws-ovn-upgrade-ipsec
ci/prow/e2e-vsphere-ovn f3d55af link false /test e2e-vsphere-ovn
ci/prow/4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade f3d55af link true /test 4.16-upgrade-from-stable-4.15-e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-aws-ovn-serial f3d55af link false /test e2e-aws-ovn-serial
ci/prow/e2e-aws-live-migration-sdn-ovn-rollback f3d55af link false /test e2e-aws-live-migration-sdn-ovn-rollback

Full PR test history. Your PR dashboard.

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.

@danwinship danwinship added the jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. label Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.