Revert "Implemented HasInstance for GCE provider" as it is causing CA E2E test failures#9470
Conversation
This reverts commit 785b523.
|
Skipping CI for Draft Pull Request. |
|
/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 |
1 similar comment
|
/test pull-autoscaling-e2e-gci-gce-ca-test |
|
/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 |
|
Looks like this is improving the stability of the tests, so let's revert and then investigate what exactly is the source of flakiness in the original PR. Right now other PRs are being blocked by flaky e2e tests. /lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Choraden, x13n The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |

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.: