-
Notifications
You must be signed in to change notification settings - Fork 2.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
Drop MULTIZONE CI jobs #21777
Drop MULTIZONE CI jobs #21777
Conversation
/assign |
I remembered the test was introduced initially as a regression tests, but I couldn't trace down the details. cc/ @BenTheElder On another hand since the CI jobs failed for a long time without much attention, I am totally ok to remove it from OSS dashboard. |
Quickly sync up with @MrHohn. The test was added to capture issues (such as misconfigured LB, etc.) when running cluster in multiple zones initially. But we all agreed that the CI is still valid for OSS K8s. Meanwhile, the initial test cases were migrated to use internal CIs already. |
+1, the ingress-gce one was added to cover a regression when cluster runs on multiple zones. I believe we have internal CIs that cover the same cases already, so I'm fine with removing this CI. |
/lgtm Thanks for cleaning up this. |
/retest |
|
Signed-off-by: Davanum Srinivas <davanum@gmail.com>
424bb70
to
75fa691
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dchen1107, dims 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:
Approvers can indicate their approval by writing |
@BenTheElder fixed now. |
/lgtm |
@dims: Updated the
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/test-infra repository. |
Drops CI jobs mentioned in #21655
Signed-off-by: Davanum Srinivas davanum@gmail.com