Skip to content

[Core] Remove the obsolete VLLM_USE_RAY_V2_EXECUTOR_BACKEND env var - #62228

Merged
robertgshaw2-redhat merged 2 commits into
mainfrom
rm-env-var
Oct 19, 2026
Merged

robertgshaw2-redhat merged 2 commits into
mainfrom
rm-env-var

Conversation

@njhill

@njhill njhill commented Oct 17, 2026

Copy link
Copy Markdown
Member

Purpose

Following #61712 (removal of the legacy Ray compiled-graph executor) and #61713, RayExecutorV2 is the only Ray executor and VLLM_USE_RAY_V2_EXECUTOR_BACKEND is ignored. This removes the env var. No in-tree code reads it.

Test Plan

Pre-commit (ruff, mypy) passes; no behavior change.

🤖 Generated with Claude Code

RayExecutorV2 is now the only Ray executor following #61712 and #61713,
so this flag is ignored. Remove it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: Nick Hill <nickhill123@gmail.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@njhill njhill added the ready ONLY add when PR is ready to merge/full CI is needed label Oct 17, 2026
@njhill

njhill commented Oct 17, 2026

Copy link
Copy Markdown
Member Author

/ci run

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #95625 for commit 13b3c44265d2.

@robertgshaw2-redhat
robertgshaw2-redhat enabled auto-merge (squash) October 17, 2026 22:37
@njhill

njhill commented Oct 18, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ Queued 3 failed job(s) for retry in Buildkite CI #95625.

@njhill

njhill commented Oct 18, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ Queued 3 failed job(s) for retry in Buildkite CI #95625.

@njhill

njhill commented Oct 18, 2026

Copy link
Copy Markdown
Member Author

/ci retry

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #95745 for commit 8d3cedfb61f5, running 3 failed step(s) from Buildkite CI #95625.

@robertgshaw2-redhat
robertgshaw2-redhat merged commit 52e88e2 into main Oct 19, 2026
23 checks passed
@robertgshaw2-redhat
robertgshaw2-redhat deleted the rm-env-var branch October 19, 2026 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants