ovnk, virt, periodics: add single stack IPv4/v6 jobs #69748

maiqueb · 2025-09-26T11:53:03Z

No description provided.

maiqueb · 2025-09-26T11:56:22Z

@JoelSpeed adding the lanes you've requested in openshift/api#2496

@everettraven FYI

maiqueb · 2025-09-26T12:14:08Z

/pj-rehearse

openshift-ci-robot · 2025-09-26T12:14:10Z

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

maiqueb · 2025-09-26T15:25:51Z

/pj-rehearse e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv4 e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6 e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6-techpreview

openshift-ci-robot · 2025-09-26T15:25:53Z

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

openshift-ci-robot · 2025-09-26T15:29:42Z

@maiqueb: job(s): e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv4, e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6, e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6-techpreview either don't exist or were not found to be affected, and cannot be rehearsed

maiqueb · 2025-09-26T15:40:21Z

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

openshift-ci-robot · 2025-09-26T15:40:23Z

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

qinqon · 2025-09-29T08:35:40Z

@maiqueb isn't the GA job need to be 4.20 instead of 4.21 ?

maiqueb · 2025-09-29T09:05:57Z

/pj-rehearse

openshift-ci-robot · 2025-09-29T09:05:59Z

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

qinqon

We don't need the extra interface at nodes since we are not testing localnet at origin, let's remove it can cause connectivity issues since as part of it a extra default gw and dns server is configured, libvirt provide those as part of DHCP answer.

qinqon · 2025-09-29T10:30:09Z

...erator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-release-4.21__periodics.yaml

+      EXTRA_NETWORK_CONFIG: |
+        EXTRA_NETWORK_NAMES="extranet"
+        EXTRANET_NETWORK_SUBNET_V4='192.168.221.0/24'


We are not testing localnet so we don't need any of those, let's remove it so we have less stuff going on at the udnerlay

qinqon · 2025-09-29T10:30:29Z

...erator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-release-4.21__periodics.yaml

+      EXTRA_NETWORK_CONFIG: |
+        EXTRA_NETWORK_NAMES="extranet"
+        EXTRANET_NETWORK_SUBNET_V4='192.168.221.0/24'


qinqon · 2025-09-29T10:31:02Z

...erator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-release-4.21__periodics.yaml

+      EXTRA_NETWORK_CONFIG: |
+        EXTRA_NETWORK_NAMES="extranet"
+        EXTRANET_NETWORK_SUBNET_V6='fd2e:6f44:5dd8:ca56::/120'


Ditto also the error that we have with installing openshift virt can be related to the fact that an extra default gw is being configured if we do this.

maiqueb · 2025-09-29T11:55:45Z

/pj-rehearse

openshift-ci-robot · 2025-09-29T11:55:48Z

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

openshift-ci · 2025-09-30T11:39:22Z

@maiqueb: 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/rehearse/periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6	`96329d7`	link	unknown	`/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6`
ci/rehearse/periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6-techpreview	`96329d7`	link	unknown	`/pj-rehearse periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv6-techpreview`

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.

maiqueb · 2025-09-30T11:47:04Z

/pj-rehearse

openshift-ci-robot · 2025-09-30T11:47:06Z

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

maiqueb · 2025-09-30T13:57:34Z

@kyrtapz both rehearsals passed. We're all set for ipv4.

Could you review ?

kyrtapz · 2025-09-30T16:30:30Z

...erator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-release-4.21__periodics.yaml

      FEATURE_SET: TechPreviewNoUpgrade
    workflow: baremetalds-e2e-ovn-bgp-virt-dualstack
  timeout: 6h0m0s
+- as: e2e-metal-ipi-ovn-bgp-virt-single-stack-ipv4-techpreview


NIT: I don't think we usually name the jobs single-stack as it is the default, same with ipv4.
What do you think about going with e2e-metal-ipi-ovn-bgp-virt-techpreview and e2e-metal-ipi-ovn-bgp-virt?

I'm confused since this is the only comment I got, but is a nit. Should I do comply with this request or ignore it ?

I don't care that much about the name, and I'd rather do my little bit to save the rain forest a lil' bit by not requiring another CI round on this, but I'll blindly trust the maintainers judgement .

Let's just rename it please to keep it consistent.
We don't have to wait for another rehearsal.

maiqueb · 2025-09-30T16:30:53Z

/pj-rehearse ack

openshift-ci-robot · 2025-09-30T16:30:55Z

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

Signed-off-by: Miguel Duarte Barroso <[email protected]>

… layered product Signed-off-by: Miguel Duarte Barroso <[email protected]>

openshift-ci-robot · 2025-09-30T17:17:12Z

[REHEARSALNOTIFIER]
@maiqueb: 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	N/A	periodic	Periodic changed
periodic-ci-openshift-ovn-kubernetes-release-4.21-periodics-e2e-metal-ipi-ovn-bgp-virt-ipv4-techpreview	N/A	periodic	Periodic 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.

kyrtapz · 2025-09-30T17:26:16Z

/lgtm

openshift-ci · 2025-09-30T17:29:57Z

[APPROVALNOTIFIER] This PR is APPROVED

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

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:

~~ci-operator/config/openshift/ovn-kubernetes/OWNERS~~ [kyrtapz]
~~ci-operator/jobs/openshift/ovn-kubernetes/OWNERS~~ [kyrtapz]

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

maiqueb · 2025-09-30T17:51:45Z

/pj-rehearse ack

openshift-ci-robot · 2025-09-30T17:51:48Z

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

openshift-ci bot requested review from jcaamano and kyrtapz September 26, 2025 11:54

maiqueb force-pushed the ovn-kubernetes-add-virt-metal-virt-jobs-single-stack branch from ff403a0 to 491ad27 Compare September 29, 2025 09:05

qinqon reviewed Sep 29, 2025

View reviewed changes

maiqueb force-pushed the ovn-kubernetes-add-virt-metal-virt-jobs-single-stack branch from 96329d7 to b556de9 Compare September 30, 2025 11:20

kyrtapz reviewed Sep 30, 2025

View reviewed changes

openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Sep 30, 2025

maiqueb added 2 commits September 30, 2025 18:14

ovnk, virt, periodics: add single stack IPv4/v6 jobs

000d322

Signed-off-by: Miguel Duarte Barroso <[email protected]>

ovnk, virt, periodics: use 4.99 CNV since 4.21 is not out yet on this…

3752514

… layered product Signed-off-by: Miguel Duarte Barroso <[email protected]>

maiqueb force-pushed the ovn-kubernetes-add-virt-metal-virt-jobs-single-stack branch from b556de9 to 3752514 Compare September 30, 2025 17:14

openshift-ci-robot removed the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Sep 30, 2025

openshift-ci bot assigned kyrtapz Sep 30, 2025

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 30, 2025

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 30, 2025

openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Sep 30, 2025

openshift-merge-bot bot merged commit 6818896 into openshift:master Sep 30, 2025
15 checks passed

ovnk, virt, periodics: add single stack IPv4/v6 jobs #69748

ovnk, virt, periodics: add single stack IPv4/v6 jobs #69748

Conversation

maiqueb commented Sep 26, 2025

Uh oh!

maiqueb commented Sep 26, 2025

Uh oh!

maiqueb commented Sep 26, 2025

Uh oh!

openshift-ci-robot commented Sep 26, 2025

Uh oh!

maiqueb commented Sep 26, 2025

Uh oh!

openshift-ci-robot commented Sep 26, 2025

Uh oh!

openshift-ci-robot commented Sep 26, 2025

Uh oh!

maiqueb commented Sep 26, 2025

Uh oh!

openshift-ci-robot commented Sep 26, 2025

Uh oh!

qinqon commented Sep 29, 2025

Uh oh!

maiqueb commented Sep 29, 2025

Uh oh!

openshift-ci-robot commented Sep 29, 2025

Uh oh!

qinqon left a comment

Choose a reason for hiding this comment

Uh oh!

qinqon Sep 29, 2025

Choose a reason for hiding this comment

Uh oh!

qinqon Sep 29, 2025

Choose a reason for hiding this comment

Uh oh!

qinqon Sep 29, 2025

Choose a reason for hiding this comment

Uh oh!

maiqueb commented Sep 29, 2025

Uh oh!

openshift-ci-robot commented Sep 29, 2025

Uh oh!

openshift-ci bot commented Sep 30, 2025

Uh oh!

maiqueb commented Sep 30, 2025

Uh oh!

openshift-ci-robot commented Sep 30, 2025

Uh oh!

maiqueb commented Sep 30, 2025

Uh oh!

kyrtapz Sep 30, 2025

Choose a reason for hiding this comment

Uh oh!

maiqueb Sep 30, 2025

Choose a reason for hiding this comment

Uh oh!

kyrtapz Sep 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

maiqueb commented Sep 30, 2025

Uh oh!

openshift-ci-robot commented Sep 30, 2025

Uh oh!

openshift-ci-robot commented Sep 30, 2025

Uh oh!

kyrtapz commented Sep 30, 2025

Uh oh!

openshift-ci bot commented Sep 30, 2025

Uh oh!

maiqueb commented Sep 30, 2025

Uh oh!

openshift-ci-robot commented Sep 30, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

kyrtapz Sep 30, 2025 •

edited

Loading