Skip to content

Commit cef2858

Browse files
test: skiping multiple va saturation test (#934)
* skip multiple va test Signed-off-by: Lionel Villard <villard@us.ibm.com> * remove Focus Signed-off-by: Lionel Villard <villard@us.ibm.com> --------- Signed-off-by: Lionel Villard <villard@us.ibm.com>
1 parent 834af86 commit cef2858

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

test/e2e/saturation_test.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -331,6 +331,7 @@ var _ = Describe("Saturation Mode - Multiple VariantAutoscalings", Label("full")
331331
)
332332

333333
BeforeAll(func() {
334+
334335
// Note: InferencePools should already exist from infra-only deployment
335336
// We no longer create InferencePools in individual tests
336337

@@ -423,6 +424,10 @@ var _ = Describe("Saturation Mode - Multiple VariantAutoscalings", Label("full")
423424
_ = k8sClient.AppsV1().Deployments(cfg.LLMDNamespace).Delete(ctx, modelServiceB+"-decode", metav1.DeleteOptions{})
424425
})
425426

427+
BeforeEach(func() {
428+
Skip("Multi-variant saturation test is currently disabled due to instability and long execution time. Re-enable after addressing underlying issues.")
429+
})
430+
426431
It("should prefer cheaper variant (VA A) for scale-up when both variants are available", func() {
427432
By("Generating load to both services")
428433
// Use burst load (curl) instead of guidellm because the simulator only tracks

0 commit comments

Comments
 (0)