origin icon indicating copy to clipboard operation
origin copied to clipboard

SDN-5202: test, network-segmentation: Add e2e tests for OVN-Kubernetes UDN CRD

Open ormergi opened this issue 1 year ago • 35 comments

This PR adds e2e tests for OVN-Kuberentes user-defined-network CRD, introduced by https://github.com/ovn-org/ovn-kubernetes/pull/4486.

Note to reviewer: The added tests ported from OVN-K repo as it, no logical change has been done.

ormergi avatar Aug 11 '24 14:08 ormergi

Hi @ormergi. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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[bot] avatar Aug 11 '24 14:08 openshift-ci[bot]

/ok-to-test

tssurya avatar Aug 12 '24 05:08 tssurya

/lgtm

npinaeva avatar Aug 13 '24 13:08 npinaeva

Job Failure Risk Analysis for sha: b95755371af4aeb173db25f473199fdc16ed473d

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-metal-ipi-ovn-kube-apiserver-rollout High
[sig-api-machinery] disruption/cache-kube-api connection/reused should be available throughout the test
This test has passed 99.22% of 128 runs on release 4.18 [Architecture:amd64 FeatureSet:default Installer:ipi Network:ovn NetworkStack:ipv4 Platform:metal SecurityMode:default Topology:ha Upgrade:none] in the last week.
---
[sig-api-machinery] disruption/cache-openshift-api connection/reused should be available throughout the test
This test has passed 99.22% of 128 runs on release 4.18 [Architecture:amd64 FeatureSet:default Installer:ipi Network:ovn NetworkStack:ipv4 Platform:metal SecurityMode:default Topology:ha Upgrade:none] in the last week.
pull-ci-openshift-origin-master-e2e-aws-ovn-single-node-serial High
[sig-storage] CSI Mock selinux on mount metrics SELinuxMount metrics [LinuxOnly] [Feature:SELinux] [Serial] warning is bumped on two Pods with a different context on RWO volume [FeatureGate:SELinuxMountReadWriteOncePod] [Beta] [Feature:SELinuxMountReadWriteOncePodOnly] [Suite:openshift/conformance/serial] [Suite:k8s]
This test has passed 100.00% of 29 runs on jobs ['periodic-ci-openshift-release-master-nightly-4.18-e2e-aws-ovn-single-node-serial'] in the last 14 days.
---
[sig-storage] CSI Mock selinux on mount metrics SELinuxMount metrics [LinuxOnly] [Feature:SELinux] [Serial] error is bumped on two Pods with a different context on RWOP volume [FeatureGate:SELinuxMountReadWriteOncePod] [Beta] [Suite:openshift/conformance/serial] [Suite:k8s]
This test has passed 100.00% of 29 runs on jobs ['periodic-ci-openshift-release-master-nightly-4.18-e2e-aws-ovn-single-node-serial'] in the last 14 days.
pull-ci-openshift-origin-master-e2e-aws-ovn-serial High
[sig-storage] CSI Mock selinux on mount metrics SELinuxMount metrics [LinuxOnly] [Feature:SELinux] [Serial] error is bumped on two Pods with a different context on RWOP volume [FeatureGate:SELinuxMountReadWriteOncePod] [Beta] [Suite:openshift/conformance/serial] [Suite:k8s]
This test has passed 100.00% of 31 runs on jobs ['periodic-ci-openshift-release-master-ci-4.18-e2e-aws-ovn-serial' 'periodic-ci-openshift-release-master-nightly-4.18-e2e-aws-ovn-serial'] in the last 14 days.
---
[sig-storage] CSI Mock selinux on mount metrics SELinuxMount metrics [LinuxOnly] [Feature:SELinux] [Serial] warning is bumped on two Pods with a different context on RWO volume [FeatureGate:SELinuxMountReadWriteOncePod] [Beta] [Feature:SELinuxMountReadWriteOncePodOnly] [Suite:openshift/conformance/serial] [Suite:k8s]
This test has passed 100.00% of 31 runs on jobs ['periodic-ci-openshift-release-master-ci-4.18-e2e-aws-ovn-serial' 'periodic-ci-openshift-release-master-nightly-4.18-e2e-aws-ovn-serial'] in the last 14 days.
pull-ci-openshift-origin-master-e2e-metal-ipi-ovn-ipv6 Low
[sig-network][OCPFeatureGate:NetworkSegmentation][Feature:UserDefinedPrimaryNetworks] EndpointSlices mirroring when using openshift ovn-kubernetes mirrors EndpointSlices managed by the default controller for namespaces with user defined primary networks L3 dualstack primary UDN, host-networked pods [Suite:openshift/conformance/parallel]
This test has passed 0.00% of 19 runs on jobs ['periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-ipv6'] in the last 14 days.
---
[sig-network][OCPFeatureGate:NetworkSegmentation][Feature:UserDefinedPrimaryNetworks] EndpointSlices mirroring when using openshift ovn-kubernetes mirrors EndpointSlices managed by the default controller for namespaces with user defined primary networks L2 dualstack primary UDN, host-networked pods [Suite:openshift/conformance/parallel]
This test has passed 0.00% of 19 runs on jobs ['periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-ipv6'] in the last 14 days.

openshift-trt-bot avatar Aug 13 '24 14:08 openshift-trt-bot

/hold land https://github.com/openshift/origin/pull/28997 first and run payloads with feature ON

tssurya avatar Aug 13 '24 17:08 tssurya

@ormergi https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/28995/pull-ci-openshift-origin-master-verify/1823292315985252352 verify is failing...

tssurya avatar Aug 13 '24 17:08 tssurya

PR 28997 has merged but please fix CI on this and also run TP+Metal lane on this PR

tssurya avatar Aug 14 '24 04:08 tssurya

New changes are detected. LGTM label has been removed.

openshift-ci[bot] avatar Aug 14 '24 09:08 openshift-ci[bot]

Changes: updated generated test-report manifest and annotations

ormergi avatar Aug 14 '24 09:08 ormergi

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: npinaeva, ormergi Once this PR has been reviewed and has the lgtm label, please assign sosiouxme for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

openshift-ci[bot] avatar Aug 14 '24 09:08 openshift-ci[bot]

@ormergi: This pull request references SDN-5202 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.18.0" version, but no target version was set.

In response to this:

This PR adds e2e tests for OVN-Kuberentes user-defined-network CRD, introduced by https://github.com/ovn-org/ovn-kubernetes/pull/4486.

Note to reviewer: The added tests ported from OVN-K repo as it, no logical change has been done.

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 avatar Aug 14 '24 09:08 openshift-ci-robot

Removing hold because https://github.com/openshift/origin/pull/28997 is merged

/unhold

ormergi avatar Aug 14 '24 09:08 ormergi

PR 28997 has merged but please fix CI on this and also run TP+Metal lane on this PR

@tssurya I am not sure which one is TP / Metal, could you please specify the job names?

ormergi avatar Aug 14 '24 09:08 ormergi

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-ipv6 openshift/api#1997 /payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm openshift/api#1997

ormergi avatar Aug 15 '24 13:08 ormergi

@ormergi: given command is invalid: at least one of the commands given is only supported on a one-command-per-comment basis, please separate out commands as multiple comments

openshift-ci[bot] avatar Aug 15 '24 13:08 openshift-ci[bot]

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-ipv6 https://github.com/openshift/api/pull/1997

ormergi avatar Aug 15 '24 13:08 ormergi

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm https://github.com/openshift/api/pull/1997

ormergi avatar Aug 15 '24 13:08 ormergi

@ormergi: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

openshift-ci[bot] avatar Aug 15 '24 13:08 openshift-ci[bot]

@ormergi: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

openshift-ci[bot] avatar Aug 15 '24 13:08 openshift-ci[bot]

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm https://github.com/openshift/api/pull/1997

ormergi avatar Aug 15 '24 13:08 ormergi

@ormergi: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

openshift-ci[bot] avatar Aug 15 '24 13:08 openshift-ci[bot]

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm openshift/api#1997

ormergi avatar Aug 15 '24 13:08 ormergi

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

  • periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/09943ab0-5b08-11ef-9839-571e36a6895a-0

openshift-ci[bot] avatar Aug 15 '24 13:08 openshift-ci[bot]

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-ipv6 openshift/api#1997

ormergi avatar Aug 15 '24 13:08 ormergi

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

  • periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-ipv6

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/5fb66670-5b08-11ef-9ac7-70a30b4950ee-0

openshift-ci[bot] avatar Aug 15 '24 13:08 openshift-ci[bot]

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm https://github.com/openshift/api/pull/1997

ormergi avatar Aug 18 '24 15:08 ormergi

@ormergi: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info.

openshift-ci[bot] avatar Aug 18 '24 15:08 openshift-ci[bot]

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm openshift/api#1997

ormergi avatar Aug 18 '24 15:08 ormergi

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

  • periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-bm

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/d1d45330-5d73-11ef-9a7f-e73dabc2a915-0

openshift-ci[bot] avatar Aug 18 '24 15:08 openshift-ci[bot]

/payload-job-with-prs periodic-ci-openshift-release-master-nightly-4.18-e2e-metal-ipi-ovn-ipv6 openshift/api#1997

ormergi avatar Aug 18 '24 15:08 ormergi