-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SDN-5473: OVN-K: Port e2e tests for CUDN functionality, connectivity and isolation #29274
Conversation
Skipping CI for Draft Pull Request. |
@ormergi: This pull request references SDN-5473 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:
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. |
/test ? |
@ormergi: The following commands are available to trigger required jobs:
The following commands are available to trigger optional jobs:
Use
In response to this:
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. |
/test e2e-gcp-ovn-techpreview |
5c62235
to
aa8ec52
Compare
@ormergi: This pull request references SDN-5473 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:
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. |
/test e2e-gcp-ovn-techpreview |
your tests are not registered with the label before make update
after make update
|
aa8ec52
to
537c033
Compare
@jluhrsen thank you! |
/test e2e-gcp-ovn-techpreview |
@ormergi: This pull request references SDN-5473 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:
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. |
/test e2e-gcp-ovn-techpreview |
e068e62
to
e3b7172
Compare
/test e2e-gcp-ovn-techpreview |
e3b7172
to
d7cb6d6
Compare
/test e2e-gcp-ovn-techpreview |
d7cb6d6
to
93de92f
Compare
/test e2e-gcp-ovn-techpreview |
1 similar comment
/test e2e-gcp-ovn-techpreview |
2d97138
to
5bbf48f
Compare
@ormergi: This pull request references SDN-5473 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:
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. |
/test e2e-gcp-ovn-techpreview |
@ormergi: No presubmit jobs available for openshift/origin@master In response to this:
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. |
/test e2e-gcp-ovn-techpreview |
/tide refresh |
Job Failure Risk Analysis for sha: 5ceef38
|
Job Failure Risk Analysis for sha: 5ceef38
|
/test e2e-aws-ovn-serial |
Job Failure Risk Analysis for sha: 5ceef38
|
/test e2e-aws-ovn-serial |
Job Failure Risk Analysis for sha: 5ceef38
|
/test e2e-aws-ovn-serial |
/tide refresh |
/test e2e-aws-ovn-serial |
/override ci/prow/e2e-aws-ovn-serial |
@dgoodwin: Overrode contexts on behalf of dgoodwin: ci/prow/e2e-aws-ovn-serial In response to this:
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. |
/tide refresh |
/test e2e-gcp-ovn-upgrade |
/override ci/prow/e2e-gcp-ovn-upgrade |
@dgoodwin: Overrode contexts on behalf of dgoodwin: ci/prow/e2e-gcp-ovn-upgrade In response to this:
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. |
/tide refresh |
/skip Just making sure all the failed ones are optional |
/override ci/prow/e2e-aws-ovn-serial |
@stbenjam: Overrode contexts on behalf of stbenjam: ci/prow/e2e-aws-ovn-serial In response to this:
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. |
/tide refresh |
Job Failure Risk Analysis for sha: 5ceef38
|
Job Failure Risk Analysis for sha: 5ceef38
|
/test e2e-metal-ipi-ovn-ipv6 |
@ormergi: all tests passed! 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. |
[ART PR BUILD NOTIFIER] Distgit: openshift-enterprise-tests |
/test e2e-aws-ovn-serial |
This PR adds e2e tests for OVN-Kuberentes CluserUserDefinedNetwork CRD, introduced by ovn-kubernetes/ovn-kubernetes#4612.
Note to reviewer:
Tests has been added as is from U/S repo, no logical changes has been done (1st & 2nd commits)
Any change required on those bits, require introducing the change on U/S and replicating these changes to D/S, after this PR is merged. In order to avoid diverging from U/S code.
Due to flakes surfaced on OCP CI the some changes has been done on U/S e2e, CUDN: Improve stability ovn-kubernetes/ovn-kubernetes#4842, this PR changes were replicated to this PR as is (3-5 commits).
Any change required on those bits, require introducing the change on the PR on U/S and replicating these changes to this PR. In order to avoid diverging from U/S code.
The changes has been done:
- Add missing wait for CUDN status to populate.
- Since tests runs in parallel on TP lane, wait for CUDN objects to be gone when deleted.
- Prevent flakes when CUDN realted tests run in parallel by using random metadata.name for CUDN objects.
In tests that create namespaces and pods, set the pods with PSA that meet the cluster.
Following incoming changes related to condition renaming the new CUDN tests were adjusted (last commit), similar to what we did at SDN-4930: networksegmentation, udn: Wait for NetworkCreated or NetworkReady condition #29381
This PR depends on having the CRD deployed by CNO SDN-5472: bindata, ovn-k: Add ClusterUserDefinedNetwork CRD and RBAC cluster-network-operator#2558