Skip to content

Conversation

Gregory-Pereira
Copy link

@Gregory-Pereira Gregory-Pereira commented Oct 20, 2025

Fix: arm wheels tagged incorrectly

Related to: https://vllm-dev.slack.com/archives/C07R5PAL2L9/p1760721161081999

Issue

The vLLM wheels index contains both arm64 and amd64 wheels for CUDA, however the wheel for arm does not contain the cu<CUDA_VERSION> identifier, while the amd64 one is.

Cause

I believe this is happening because in the arm wheel pipeline we pass VLLM_MAIN_CUDA_VERSION as 12.9 which it understands to be the same Major and Minor version as CUDA_VERSION of 12.9.1 for this pipeline - as a result it omits the cu129 identifier from the build. This should fix our release pipeline to ensure wheels match the index generated for them and are consistent with the x86_64 ones.

cc @wseaton @simon-mo @khluu

@github-actions
Copy link

👋 Hi! Thank you for contributing to the vLLM project.

💬 Join our developer Slack at https://slack.vllm.ai to discuss your PR in #pr-reviews, coordinate on features in #feat- channels, or join special interest groups in #sig- channels.

Just a reminder: PRs would not trigger full CI run by default. Instead, it would only run fastcheck CI which starts running only a small and essential subset of CI tests to quickly catch errors.

You ask your reviewers to trigger select CI tests on top of fastcheck CI.

Once the PR is approved and ready to go, your PR reviewer(s) can run CI to test the changes comprehensively before merging.

To run CI, PR reviewers can either: Add ready label to the PR or enable auto-merge.

If you have any questions, please reach out to us on Slack at https://slack.vllm.ai.

🚀

@Gregory-Pereira Gregory-Pereira changed the title passthrough VLLM_MAIN_CUDA_VERSION to base image and wheel build stages ARM64 CUDA 12.9 wheels built and uploaded to index incorrectly Oct 20, 2025
@mergify mergify bot added the ci/build label Oct 20, 2025
@Gregory-Pereira Gregory-Pereira marked this pull request as ready for review October 20, 2025 22:42
@Gregory-Pereira Gregory-Pereira force-pushed the fix-vLLM-arm-wheel-links branch from cd2495d to f564496 Compare October 21, 2025 16:01
@Gregory-Pereira Gregory-Pereira force-pushed the fix-vLLM-arm-wheel-links branch from f564496 to cd1f381 Compare October 21, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant