-
Notifications
You must be signed in to change notification settings - Fork 12
Adapt egressIP test to dualstack #200
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
Adapt egressIP test to dualstack #200
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/test build |
|
/cc @MiguelCarpio |
|
@MaysaMacedo is there a job we could trigger here to test the result? |
KURYRQE-1414 While running openstack-test in ipv6primary dualstack cluster, there are some tests that are not passing. This patch performs the needed actions to make those tests available in both ipv4 and ipv6 primary dualstack cluster.
SOSQE-1388 SOSQE-217 - Adapting existing ipv4 test to run in all type of dualstack clusters. - Including test to cover OCPBUGS-27222
a7b10e0 to
1950980
Compare
|
/test test |
|
@EmilienM there is no job yet. I'm proposing it here openshift/release#59211 |
|
/retest-required |
1 similar comment
|
/retest-required |
|
Is this for https://issues.redhat.com/browse/OCPBUGS-42498 ? |
@mandre Sort of, it's related to the same test case that it's failing, but it's for the dualstack job and not openstack-ccpmso-zone. |
|
/test e2e-openstack-dualstack-techpreview |
1 similar comment
|
/test e2e-openstack-dualstack-techpreview |
|
/retest-required |
|
/test e2e-openstack-dualstack-techpreview |
2 similar comments
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
|
@MaysaMacedo @mandre To pass the test we need to resolve this first https://issues.redhat.com/browse/OCPBUGS-43496 |
|
@MaysaMacedo Why do you want to backport these fixes? 4.14 does not officially support dualstack, right? |
|
/test e2e-openstack-dualstack-techpreview |
4 similar comments
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
4 similar comments
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
|
/test e2e-openstack-dualstack-techpreview |
|
@MaysaMacedo: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
|
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
|
Rotten issues close after 30d of inactivity. Reopen the issue by commenting /close |
|
@openshift-bot: Closed this PR. DetailsIn 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. |
|
I opened a specific bug for this OCPBUGS-59907 |
This PR backports the following two commits:
isDualStackClusterfunction needed by the next commitgetEgressIPNetwork(primaryWorker)togetEgressNetworkInfo(primaryWorker, "ipv4")which returns the specific Port to be used and should help with the error seen in the dualstack jobs:https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-ci-shiftstack-ci-release-4.14-e2e-openstack-dualstack-techpreview/1849493935328595968