Skip to content

Conversation

@ormergi
Copy link
Contributor

@ormergi ormergi commented Dec 18, 2025

Some ovn-k bgp virt periodic jobs on release-4.21 installed latest version of CNV which is no longer relevent following recent 4.22 cut-off.

This PR change the jobs config to install correct CNV version - 4.21.
The affected jobs are the following release-4.21 periodic jobs:

  • e2e-metal-ipi-ovn-bgp-virt-dualstack
  • e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview
  • e2e-metal-ipi-ovn-bgp-virt-ipv4
  • e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview

@openshift-ci openshift-ci bot requested review from jcaamano and kyrtapz December 18, 2025 14:17
@ormergi ormergi changed the title ovn-k,bgp,virt,periodics,4.21: Pin CNV version to OCP version OCPBUGS-69826: ovn-k,bgp,virt,periodics,4.21: Pin CNV version to OCP version Dec 18, 2025
@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Dec 18, 2025
@ormergi
Copy link
Contributor Author

ormergi commented Dec 18, 2025

/cc @maiqueb

@openshift-ci-robot
Copy link
Contributor

@ormergi: This pull request references Jira Issue OCPBUGS-69826, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.22.0) matches configured target version for branch (4.22.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @anuragthehatter

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

Some ovn-k bgp virt periodic jobs on release-4.21 installed latest version of CNV which is no longer relevent following recent 4.22 cut-off.

This PR change the jobs config to install correct CNV version - 4.21.
The affected jobs are the following release-4.21 periodic jobs:

  • e2e-metal-ipi-ovn-bgp-virt-dualstack
  • e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview
  • e2e-metal-ipi-ovn-bgp-virt-ipv4
  • e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview

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.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 18, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 18, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.20-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 18, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.20-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 18, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.20-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci-robot
Copy link
Contributor

@ormergi: job(s): periodic-ci-openshift-ovn-kubernetes-release-4.20-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview either don't exist or were not found to be affected, and cannot be rehearsed

@openshift-ci-robot
Copy link
Contributor

@ormergi: job(s): periodic-ci-openshift-ovn-kubernetes-release-4.20-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview either don't exist or were not found to be affected, and cannot be rehearsed

@openshift-ci-robot
Copy link
Contributor

@ormergi: job(s): periodic-ci-openshift-ovn-kubernetes-release-4.20-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4 either don't exist or were not found to be affected, and cannot be rehearsed

@maiqueb
Copy link
Contributor

maiqueb commented Dec 18, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4

@openshift-ci-robot
Copy link
Contributor

@maiqueb: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 18, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 18, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 19, 2025

All affected jobs install the correct CNV version
/pj-rehearse ack

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Dec 19, 2025
@ormergi
Copy link
Contributor Author

ormergi commented Dec 19, 2025

@maiqueb @kyrtapz PTAL

@openshift-ci-robot
Copy link
Contributor

[REHEARSALNOTIFIER]
@ormergi: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview N/A periodic Ci-operator config changed
periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack N/A periodic Ci-operator config changed
periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview N/A periodic Ci-operator config changed
periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4 N/A periodic Ci-operator config changed
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 19, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 19, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 19, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@ormergi
Copy link
Contributor Author

ormergi commented Dec 19, 2025

/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-dualstack

@openshift-ci-robot
Copy link
Contributor

@ormergi: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

Copy link
Contributor

@maiqueb maiqueb left a comment

Choose a reason for hiding this comment

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

tks. now that we have 4.21 CNV builds, this makes sense.

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

maiqueb commented Dec 19, 2025

/verified by ci

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Dec 19, 2025
@openshift-ci-robot
Copy link
Contributor

@maiqueb: This PR has been marked as verified by @ci.

Details

In response to this:

/verified by @ci

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.

@maiqueb
Copy link
Contributor

maiqueb commented Dec 19, 2025

/verified by ci

@openshift-ci-robot
Copy link
Contributor

@maiqueb: This PR has been marked as verified by ci.

Details

In response to this:

/verified by ci

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.

@maiqueb
Copy link
Contributor

maiqueb commented Dec 19, 2025

/pj-rehearse ack

@openshift-ci-robot
Copy link
Contributor

@maiqueb: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Dec 19, 2025
@ormergi
Copy link
Contributor Author

ormergi commented Jan 4, 2026

Hi @kyrtapz / @tssurya, this is a follow up work for #72600, could you please have a look?

@tssurya
Copy link
Contributor

tssurya commented Jan 5, 2026

/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 5, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: maiqueb, ormergi, tssurya

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:

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 added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 5, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 5, 2026

@ormergi: 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.

@openshift-merge-bot openshift-merge-bot bot merged commit ebaeb42 into openshift:master Jan 5, 2026
17 checks passed
@openshift-ci-robot
Copy link
Contributor

@ormergi: Jira Issue Verification Checks: Jira Issue OCPBUGS-69826
✔️ This pull request was pre-merge verified.
✔️ All associated pull requests have merged.
✔️ All associated, merged pull requests were pre-merge verified.

Jira Issue OCPBUGS-69826 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓

Details

In response to this:

Some ovn-k bgp virt periodic jobs on release-4.21 installed latest version of CNV which is no longer relevent following recent 4.22 cut-off.

This PR change the jobs config to install correct CNV version - 4.21.
The affected jobs are the following release-4.21 periodic jobs:

  • e2e-metal-ipi-ovn-bgp-virt-dualstack
  • e2e-metal-ipi-ovn-bgp-virt-dualstack-techpreview
  • e2e-metal-ipi-ovn-bgp-virt-ipv4
  • e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview

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.

@ormergi ormergi deleted the ovn-k-bgp-virt-ipv4-pin-cnv-version-4.21 branch January 5, 2026 08:49
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. 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. rehearsals-ack Signifies that rehearsal jobs have been acknowledged verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants