Skip to content

chore: update vllm-metal version to v0.2.0 and vllm version to 0.19.1#876

Open
ilopezluna wants to merge 3 commits intomainfrom
bump-vllm-and-vllm-metal
Open

chore: update vllm-metal version to v0.2.0 and vllm version to 0.19.1#876
ilopezluna wants to merge 3 commits intomainfrom
bump-vllm-and-vllm-metal

Conversation

@ilopezluna
Copy link
Copy Markdown
Contributor

This pull request updates the vLLM Metal backend to use the latest upstream and internal release versions. The main changes are version bumps to ensure compatibility with the newest features and bug fixes.

Dependency version updates:

  • Updated VLLM_UPSTREAM_VERSION to 0.19.1 and VLLM_METAL_RELEASE to v0.2.0-20260420-142150 in .versions to align with the latest vLLM releases.
  • Updated the vllmMetalVersion constant in pkg/inference/backends/vllm/vllm_metal.go to match the new internal release tag.

vLLM: https://github.com/vllm-project/vllm/releases/tag/v0.19.1
vLLM Metal: https://github.com/vllm-project/vllm-metal/releases/tag/v0.2.0-20260420-142150

@ilopezluna ilopezluna requested a review from a team April 20, 2026 19:50
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the vLLM upstream version to 0.19.1 and the vLLM metal release to v0.2.0-20260420-142150 across the configuration and source files. A critical review comment identifies a version inconsistency where VLLM_VERSION remains at 0.19.0 despite the upstream update, which could lead to mismatched components within the system.

Comment thread .versions Outdated
@ericcurtin
Copy link
Copy Markdown
Contributor

Might be worth bumping vllm-metal to 0.19.1 before we do this. 0.19.0 might be better for this PR as .1 is not implemented in vllm-metal.

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.

2 participants