search

openshift / origin

Conformance test suite for OpenShift
http://www.openshift.org
Apache License 2.0
8.49k stars 4.7k forks source link

SDN-5346: image, mirror kubevirt fedora guest image #29229

Closed qinqon closed 3 weeks ago

qinqon commented 4 weeks ago

The ovn-kubernetes test to check persistent IPs during live migration use kubevirt to start VMs, this change add the guest image for those VMs.

Needed for:

xueqzhan commented 4 weeks ago

/lgtm

qinqon commented 4 weeks ago

@bertinatto @tkashem can you take a look ?

openshift-ci[bot] commented 3 weeks ago

@qinqon: 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-aws-ovn-single-node-serial a618a24a39a029590f4f0956237dedbe58355615 link false /test e2e-aws-ovn-single-node-serial
ci/prow/e2e-aws-ovn-single-node a618a24a39a029590f4f0956237dedbe58355615 link false /test e2e-aws-ovn-single-node
ci/prow/e2e-metal-ipi-ovn-kube-apiserver-rollout a618a24a39a029590f4f0956237dedbe58355615 link false /test e2e-metal-ipi-ovn-kube-apiserver-rollout

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
qinqon commented 3 weeks ago

/cc @bertinatto @ingvagabund @soltysh @tkashem

Can you take a view to see if it's ok for origin ? We needed this to GA a OCP feature related to ovn-kubernetes primary UDN.

openshift-ci[bot] commented 3 weeks ago

@qinqon: GitHub didn't allow me to request PR reviews from the following users: soltysh.

Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to [this](https://github.com/openshift/origin/pull/29229#issuecomment-2440656958): >/cc @bertinatto @ingvagabund @soltysh @tkashem > >Can you take a view to see if it's ok for origin ? We needed this to GA a OCP feature related to ovn-kubernetes primary UDN. > > > Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
openshift-ci[bot] commented 3 weeks ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bertinatto, maiqueb, qinqon, xueqzhan

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: - ~~[test/extended/util/image/OWNERS](https://github.com/openshift/origin/blob/master/test/extended/util/image/OWNERS)~~ [bertinatto] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
openshift-ci-robot commented 3 weeks ago

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

In response to [this](https://github.com/openshift/origin/pull/29229): >The ovn-kubernetes test to check persistent IPs during live migration use kubevirt to start VMs, this change add the guest image for those VMs. > >Needed for: >- https://github.com/openshift/origin/pull/29199 Instructions for interacting with me using PR comments are available [here](https://prow.ci.openshift.org/command-help?repo=openshift%2Forigin). If you have questions or suggestions related to my behavior, please file an issue against the [openshift-eng/jira-lifecycle-plugin](https://github.com/openshift-eng/jira-lifecycle-plugin/issues/new) repository.
maiqueb commented 3 weeks ago

@qinqon please set the SDN-5346 user story instead of no-jira.

qinqon commented 3 weeks ago

/retest-required

openshift-ci-robot commented 3 weeks ago

@qinqon: This pull request references SDN-5346 which is a valid jira issue.

In response to [this](https://github.com/openshift/origin/pull/29229): >The ovn-kubernetes test to check persistent IPs during live migration use kubevirt to start VMs, this change add the guest image for those VMs. > >Needed for: >- https://github.com/openshift/origin/pull/29199 Instructions for interacting with me using PR comments are available [here](https://prow.ci.openshift.org/command-help?repo=openshift%2Forigin). If you have questions or suggestions related to my behavior, please file an issue against the [openshift-eng/jira-lifecycle-plugin](https://github.com/openshift-eng/jira-lifecycle-plugin/issues/new) repository.
qinqon commented 3 weeks ago

/jira refresh

openshift-ci-robot commented 3 weeks ago

@qinqon: This pull request references SDN-5346 which is a valid jira issue.

In response to [this](https://github.com/openshift/origin/pull/29229#issuecomment-2449934859): >/jira refresh Instructions for interacting with me using PR comments are available [here](https://prow.ci.openshift.org/command-help?repo=openshift%2Forigin). If you have questions or suggestions related to my behavior, please file an issue against the [openshift-eng/jira-lifecycle-plugin](https://github.com/openshift-eng/jira-lifecycle-plugin/issues/new) repository.
openshift-trt-bot commented 3 weeks ago

Job Failure Risk Analysis for sha: a618a24a39a029590f4f0956237dedbe58355615

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-metal-ipi-ovn-kube-apiserver-rollout IncompleteTests
Tests for this run (24) are below the historical average (1108): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-master-e2e-metal-ipi-ovn IncompleteTests
Tests for this run (11) are below the historical average (1835): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
openshift-bot commented 3 weeks ago

[ART PR BUILD NOTIFIER]

Distgit: openshift-enterprise-tests This PR has been included in build openshift-enterprise-tests-container-v4.18.0-202411010307.p0.gb970e32.assembly.stream.el9. All builds following this will include this PR.