Skip to content

e2e test: skip test case "Replica stability under constant load" #737

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.

Sign up for GitHub

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

Open
shuynh2017 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

e2e test: skip test case "Replica stability under constant load" #737

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion test/e2e-saturation-based/e2e_saturation_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -392,7 +392,12 @@ var _ = Describe("Test workload-variant-autoscaler - Saturation Mode - Single Va
Expect(err).NotTo(HaveOccurred(), "Should be able to log VariantAutoscaling status after scale-up")
})
})


/* This test is flaky and needs more investigation, re-enable once issue is resolved. When running with
"Scale-up under load" test above, the initialReplicas can be around 2 and the currentReplicas can be 4 or 5
after 3 minutes thus failing the test. This may be due to the load generation from the previous test.
When running with just itself, the intialReplicas is 1 and the currentReplicas can be 3 thus failing the
test as well.
Context("Replica stability under constant load", func() {
It("should maintain stable replica count under constant load", func() {
By("starting constant load generation")
Expand Down Expand Up @@ -460,6 +465,7 @@ var _ = Describe("Test workload-variant-autoscaler - Saturation Mode - Single Va
Expect(err).NotTo(HaveOccurred(), "Should be able to log VariantAutoscaling status after stability check")
})
})
*/

AfterAll(func() {
By("cleaning up test resources")
Expand Down