Skip to content

deploy: default BENCHMARK_MODE to false#610

Merged
ev-shindin merged 2 commits intollm-d:mainfrom
ev-shindin:feature/benchmark-mode-default-false
Jan 20, 2026
Merged

deploy: default BENCHMARK_MODE to false#610
ev-shindin merged 2 commits intollm-d:mainfrom
ev-shindin:feature/benchmark-mode-default-false

Conversation

@ev-shindin
Copy link
Copy Markdown
Collaborator

@ev-shindin ev-shindin commented Jan 20, 2026

Summary

  • Change the default value of BENCHMARK_MODE from "true" to "false" in the install script
  • Add clarification that istioBench environment is required in helmfile when enabling benchmark mode
  • Explicitly set BENCHMARK_MODE=false in CI e2e OpenShift workflow

Background

The BENCHMARK_MODE setting switches the Istio gateway configuration to use the istioBench environment in the llm-d helmfile. However, this environment is not available in the standard llm-d helmfile, causing deployment failures when benchmark mode is enabled by default.

Changes

  1. deploy/install.sh: Changed default from "true" to "false" and updated comment to clarify the istioBench requirement
  2. .github/workflows/ci-e2e-openshift.yaml: Added explicit BENCHMARK_MODE: "false" to both Model A1 and Model B deployment steps to ensure CI reliability

Test plan

  • Verify install script deploys successfully with default settings
  • Verify CI e2e OpenShift workflow runs without benchmark mode errors
  • Verify benchmark mode can still be enabled via BENCHMARK_MODE=true when istioBench environment is available

Change the default value of BENCHMARK_MODE from "true" to "false" and
add clarification that istioBench env is required in helmfile when
enabling benchmark mode.

Also update the CI e2e OpenShift workflow to explicitly set
BENCHMARK_MODE=false since the istioBench environment is not available
in the llm-d helmfile.
@ev-shindin
Copy link
Copy Markdown
Collaborator Author

/ok-to-test

@github-actions
Copy link
Copy Markdown
Contributor

🚀 E2E tests triggered by /ok-to-test

View the OpenShift E2E workflow run

@ev-shindin ev-shindin self-assigned this Jan 20, 2026
@clubanderson
Copy link
Copy Markdown
Contributor

you are also held up by changes in #585 - you need to switch BENCHMARK_MODE default from true → false because istioBench environment doesn't exist in llm-d helm file

#603 is also suffering from this. You can include the 'BENCHMARK_MODE default from true → false' in this PR to help you move forward

@clubanderson
Copy link
Copy Markdown
Contributor

/retest

@github-actions
Copy link
Copy Markdown
Contributor

🚀 E2E tests triggered by /retest

View the OpenShift E2E workflow run

Increase requestsPerWorker from 1000 to 1100 for Model A1 and from 500
to 550 for Model B (10% increase) to sustain load slightly longer
through scale-up while staying within job completion timeout.
@ev-shindin
Copy link
Copy Markdown
Collaborator Author

/retest

@github-actions
Copy link
Copy Markdown
Contributor

🚀 E2E tests triggered by /retest

View the OpenShift E2E workflow run

@WheelyMcBones
Copy link
Copy Markdown
Collaborator

Since llm-d does not support the benchmark config for Istio anymore, the BENCHMARK_MODE might as well be deleted completely, also removing the following check:
https://github.com/llm-d-incubation/workload-variant-autoscaler/blob/d0c4cd96244bbdbe863219320146336b18955813/deploy/install.sh#L497-L501

Copy link
Copy Markdown
Collaborator

@asm582 asm582 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@ev-shindin ev-shindin merged commit 48215e5 into llm-d:main Jan 20, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants