Skip to content

Comments

CORENET-6168: skip new NetSeg Localnet test#2963

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
jluhrsen:fix-new-ote-failure
Feb 4, 2026
Merged

CORENET-6168: skip new NetSeg Localnet test#2963
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
jluhrsen:fix-new-ote-failure

Conversation

@jluhrsen
Copy link
Contributor

SetupUnderlay() is not implemented in the openshift provider API so the new test will fail in our OTE framework when it's finally enabled

📑 Description

Fixes #

Additional Information for reviewers

✅ Checks

  • My code requires changes to the documentation
  • if so, I have updated the documentation as required
  • My code requires tests
  • if so, I have added and/or updated the tests as required
  • All the tests have passed in the CI

How to verify it

SetupUnderlay() is not implemented in the openshift provider
API so the new test will fail in our OTE framework when it's
finally enabled

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
@openshift-ci openshift-ci bot requested review from jcaamano and kyrtapz January 27, 2026 23:02
@jluhrsen
Copy link
Contributor Author

/test unit

@jluhrsen
Copy link
Contributor Author

/payload-job-with-prs periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn-conformance openshift/origin#30560

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

@jluhrsen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn-conformance

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/e73834d0-fbdd-11f0-8d20-6d99c696bddd-0

@jcaamano
Copy link
Contributor

/lgtm
/approve

@openshift-ci openshift-ci bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jan 28, 2026
@jluhrsen
Copy link
Contributor Author

/retest-required

@jluhrsen jluhrsen changed the title skip new NetSeg Localnet test CORENET-6168: skip new NetSeg Localnet test Jan 28, 2026
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jan 28, 2026
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jan 28, 2026

@jluhrsen: This pull request references CORENET-6168 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 "4.22.0" version, but no target version was set.

Details

In response to this:

SetupUnderlay() is not implemented in the openshift provider API so the new test will fail in our OTE framework when it's finally enabled

📑 Description

Fixes #

Additional Information for reviewers

✅ Checks

  • My code requires changes to the documentation
  • if so, I have updated the documentation as required
  • My code requires tests
  • if so, I have added and/or updated the tests as required
  • All the tests have passed in the CI

How to verify it

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.

@jluhrsen
Copy link
Contributor Author

/payload-job-with-prs periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn openshift/origin#30560

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

@jluhrsen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/853d5aa0-fc71-11f0-8a73-66457f7c6cea-0

@jluhrsen
Copy link
Contributor Author

/payload-job-with-prs periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn openshift/origin#30560

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

@jluhrsen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/71e08120-fca4-11f0-8a54-9024b98cc770-0

@jluhrsen
Copy link
Contributor Author

/retest-required

1 similar comment
@jluhrsen
Copy link
Contributor Author

/retest-required

@jluhrsen
Copy link
Contributor Author

when I can show that this payload job below runs and actually skips the test I will add a verified label based on that:

/payload-job-with-prs periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn openshift/origin#30560

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 29, 2026

@jluhrsen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/198afb60-fd38-11f0-84df-0bd2540c29dd-0

@jcaamano
Copy link
Contributor

/override e2e-aws-ovn-fdp-qe

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 29, 2026

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

  • e2e-aws-ovn-fdp-qe

Only the following failed contexts/checkruns were expected:

  • ci/prow/4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade
  • ci/prow/4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade-ipsec
  • ci/prow/4.22-upgrade-from-stable-4.21-e2e-gcp-ovn-rt-upgrade
  • ci/prow/4.22-upgrade-from-stable-4.21-images
  • ci/prow/e2e-aws-ovn
  • ci/prow/e2e-aws-ovn-edge-zones
  • ci/prow/e2e-aws-ovn-fdp-qe
  • ci/prow/e2e-aws-ovn-hypershift
  • ci/prow/e2e-aws-ovn-local-gateway
  • ci/prow/e2e-aws-ovn-local-to-shared-gateway-mode-migration
  • ci/prow/e2e-aws-ovn-serial
  • ci/prow/e2e-aws-ovn-shared-to-local-gateway-mode-migration
  • ci/prow/e2e-aws-ovn-upgrade
  • ci/prow/e2e-aws-ovn-upgrade-local-gateway
  • ci/prow/e2e-aws-ovn-windows
  • ci/prow/e2e-azure-ovn-upgrade
  • ci/prow/e2e-gcp-ovn
  • ci/prow/e2e-gcp-ovn-techpreview
  • ci/prow/e2e-metal-ipi-ovn-dualstack
  • ci/prow/e2e-metal-ipi-ovn-dualstack-bgp
  • ci/prow/e2e-metal-ipi-ovn-dualstack-bgp-local-gw
  • ci/prow/e2e-metal-ipi-ovn-ipv6
  • ci/prow/gofmt
  • ci/prow/images
  • ci/prow/lint
  • ci/prow/okd-scos-images
  • ci/prow/qe-perfscale-aws-ovn-small-udn-density-churn-l3
  • ci/prow/qe-perfscale-aws-ovn-small-udn-density-l3
  • ci/prow/qe-perfscale-payload-control-plane-6nodes
  • ci/prow/security
  • ci/prow/unit
  • pull-ci-openshift-ovn-kubernetes-master-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade
  • pull-ci-openshift-ovn-kubernetes-master-4.22-upgrade-from-stable-4.21-e2e-aws-ovn-upgrade-ipsec
  • pull-ci-openshift-ovn-kubernetes-master-4.22-upgrade-from-stable-4.21-e2e-gcp-ovn-rt-upgrade
  • pull-ci-openshift-ovn-kubernetes-master-4.22-upgrade-from-stable-4.21-images
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-edge-zones
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-fdp-qe
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-hypershift
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-local-gateway
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-local-to-shared-gateway-mode-migration
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-serial
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-shared-to-local-gateway-mode-migration
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-upgrade
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-upgrade-local-gateway
  • pull-ci-openshift-ovn-kubernetes-master-e2e-aws-ovn-windows
  • pull-ci-openshift-ovn-kubernetes-master-e2e-azure-ovn-upgrade
  • pull-ci-openshift-ovn-kubernetes-master-e2e-gcp-ovn
  • pull-ci-openshift-ovn-kubernetes-master-e2e-gcp-ovn-techpreview
  • pull-ci-openshift-ovn-kubernetes-master-e2e-metal-ipi-ovn-dualstack
  • pull-ci-openshift-ovn-kubernetes-master-e2e-metal-ipi-ovn-dualstack-bgp
  • pull-ci-openshift-ovn-kubernetes-master-e2e-metal-ipi-ovn-dualstack-bgp-local-gw
  • pull-ci-openshift-ovn-kubernetes-master-e2e-metal-ipi-ovn-ipv6
  • pull-ci-openshift-ovn-kubernetes-master-gofmt
  • pull-ci-openshift-ovn-kubernetes-master-images
  • pull-ci-openshift-ovn-kubernetes-master-lint
  • pull-ci-openshift-ovn-kubernetes-master-okd-scos-images
  • pull-ci-openshift-ovn-kubernetes-master-qe-perfscale-aws-ovn-small-udn-density-churn-l3
  • pull-ci-openshift-ovn-kubernetes-master-qe-perfscale-aws-ovn-small-udn-density-l3
  • pull-ci-openshift-ovn-kubernetes-master-qe-perfscale-payload-control-plane-6nodes
  • pull-ci-openshift-ovn-kubernetes-master-security
  • pull-ci-openshift-ovn-kubernetes-master-unit
  • tide

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

Details

In response to this:

/override e2e-aws-ovn-fdp-qe

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.

@jluhrsen
Copy link
Contributor Author

/payload-job-with-prs periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn openshift/origin#30560

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 29, 2026

@jluhrsen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-hypershift-release-4.21-periodics-e2e-aws-ovn

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/d683e260-fd4a-11f0-8916-4ab7b520e9a6-0

@jluhrsen
Copy link
Contributor Author

/payload-job-with-prs periodic-ci-openshift-release-master-ci-4.21-e2e-aws-ovn-techpreview openshift/origin#30560

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 30, 2026

@jluhrsen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-ci-4.21-e2e-aws-ovn-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/35ec87d0-fd9a-11f0-99c1-8993f5297c75-0

@zhaozhanqi
Copy link
Contributor

/verified by ci

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jan 30, 2026
@openshift-ci-robot
Copy link
Contributor

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

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 3466e1d and 2 for PR HEAD 7dd0e55 in total

@jluhrsen
Copy link
Contributor Author

/tide refresh

@jluhrsen
Copy link
Contributor Author

jluhrsen commented Feb 2, 2026

/test e2e-aws-ovn-fdp-qe

@jluhrsen
Copy link
Contributor Author

jluhrsen commented Feb 4, 2026

/override e2e-aws-ovn-fdp-qe

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 4, 2026

@jluhrsen: jluhrsen unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers openshift-sustaining-engineers.

Details

In response to this:

/override e2e-aws-ovn-fdp-qe

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.

@jluhrsen
Copy link
Contributor Author

jluhrsen commented Feb 4, 2026

/help

@jluhrsen
Copy link
Contributor Author

jluhrsen commented Feb 4, 2026

/lgtm cancel

@jluhrsen
Copy link
Contributor Author

jluhrsen commented Feb 4, 2026

/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 4, 2026

@jluhrsen: you cannot LGTM your own PR.

Details

In response to this:

/lgtm

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 removed the lgtm Indicates that a PR is ready to be merged. label Feb 4, 2026
@abhat
Copy link
Contributor

abhat commented Feb 4, 2026

/override ci/prow/e2e-aws-ovn-fdp-qe

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 4, 2026

@abhat: Overrode contexts on behalf of abhat: ci/prow/e2e-aws-ovn-fdp-qe

Details

In response to this:

/override ci/prow/e2e-aws-ovn-fdp-qe

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.

@abhat
Copy link
Contributor

abhat commented Feb 4, 2026

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 4, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 4, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abhat, jcaamano, jluhrsen

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
Copy link
Contributor

openshift-ci bot commented Feb 4, 2026

@jluhrsen: 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/qe-perfscale-aws-ovn-small-udn-density-churn-l3 7dd0e55 link false /test qe-perfscale-aws-ovn-small-udn-density-churn-l3
ci/prow/security 7dd0e55 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.

@abhat
Copy link
Contributor

abhat commented Feb 4, 2026

/override ci/prow/e2e-metal-ipi-ovn-dualstack

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 4, 2026

@abhat: Overrode contexts on behalf of abhat: ci/prow/e2e-metal-ipi-ovn-dualstack

Details

In response to this:

/override ci/prow/e2e-metal-ipi-ovn-dualstack

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-merge-bot openshift-merge-bot bot merged commit 3cbf397 into openshift:master Feb 4, 2026
30 of 32 checks passed
@jluhrsen jluhrsen deleted the fix-new-ote-failure branch February 5, 2026 04:24
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-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants