Revert "Implemented HasInstance for GCE provider" as it is causing CA E2E test failures#9470
Revert "Implemented HasInstance for GCE provider" as it is causing CA E2E test failures#9470Choraden wants to merge 1 commit intokubernetes:masterfrom
Conversation
This reverts commit 785b523.
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Choraden 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 all |
|
The test succeeded. Rerunning to gather more datapoints. |
|
/test pull-autoscaling-e2e-gci-gce-ca-test |
1 similar comment
|
/test pull-autoscaling-e2e-gci-gce-ca-test |
|
@domenicbozzuto @jbtk @x13n Since April 3rd both CA presubmits and periodic E2E tests fail(/are flaky, but mostly fail): TBH I don't really understand how it breaks the CA, but when it comes to test cases, usually there are 3 nodes and we expect the scale up to 5. For some reason CA does 3->5 and in the next loop 5->6. |
|
Since the tests passed in the end on this PR from what I see in the comments can we run these tests 20 times with and without PR and compare flakiness? |
I believe the presubmit dashboard should provide us enough insights: https://prow.k8s.io/job-history/gs/kubernetes-ci-logs/pr-logs/directory/pull-autoscaling-e2e-gci-gce-ca-test |
|
👋 Sorry for the noise with this; I found a similar pattern when I added my PR originally (comment) -- the multiple upscales seemed exactly like the reason I'm fine with the revert if it's breaking a lot of tests, I can try to spend some time looking at why it's triggering multiple scale-ups and why that seems more common with this change. |
|
AFAIR |
|
/test pull-autoscaling-e2e-gci-gce-ca-test |
|
/test pull-autoscaling-e2e-gci-gce-ca-test |
|
/test pull-autoscaling-e2e-gci-gce-ca-test |
1 similar comment
|
/test pull-autoscaling-e2e-gci-gce-ca-test |
|
/test pull-autoscaling-e2e-gci-gce-ca-test This will be the ninth test run (so far 8 passes in a row 😊) |
|
Previous run failed due to infra errors: I'd ignore it as the infra flake. Anyway, I'm pretty confident this is the culprit. Notice that all other PRs struggle to have two successful runs while there we already have 8. Let me know if we need more proof. /test pull-autoscaling-e2e-gci-gce-ca-test |
What type of PR is this?
/kind failing-test
What this PR does / why we need it:
This reverts commit 785b523.
Since April 3rd both CA presubmits and periodic E2E tests fail:
https://prow.k8s.io/job-history/gs/kubernetes-ci-logs/pr-logs/directory/pull-autoscaling-e2e-gci-gce-ca-test
https://prow.k8s.io/job-history/gs/kubernetes-ci-logs/logs/ci-kubernetes-e2e-gci-gce-autoscaling
I have reasons to think this change is causing CA E2E tests failures.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: