Skip to content

Verify vllm-tpu python package (draft) #1241

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
ylangtsou wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Verify vllm-tpu python package (draft) #1241

ylangtsou wants to merge 3 commits into from

Conversation

@ylangtsou
Copy link
Collaborator

@ylangtsou ylangtsou commented Dec 4, 2025

Description

Verify vllm-tpu python package

Tests

Test on buildkite

Checklist

Before submitting this PR, please make sure:

  • I have performed a self-review of my code.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have made or will make corresponding changes to any relevant documentation.

@dennisYehCienet dennisYehCienet added the ready ONLY add when PR is ready to merge/full CI is needed label Dec 4, 2025
@dennisYehCienet dennisYehCienet changed the title Verify vllm-tpu python package Verify vllm-tpu python package (draft) Dec 4, 2025
@ylangtsou ylangtsou requested a review from vipannalla as a code owner December 4, 2025 08:57
@dennisYehCienet dennisYehCienet marked this pull request as draft December 5, 2025 01:41
Add Dockerfile to verify vllm-tpu wheel
87a31c2 
Signed-off-by: Ylang Tsou <ylangt@google.com>
@ylangtsou ylangtsou force-pushed the ylangt/run_with_pypi branch from 3c1e18f to 87a31c2 Compare December 8, 2025 01:44
Ylang Tsou added 2 commits December 8, 2025 10:58
fix tpu-info
8259622 
Signed-off-by: Ylang Tsou <ylangt@google.com>
fix date
a64a254 
Signed-off-by: Ylang Tsou <ylangt@google.com>
@CienetStingLin CienetStingLin added bug Something isn't working and removed bug Something isn't working labels Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jcyang43 jcyang43 Awaiting requested review from jcyang43 jcyang43 is a code owner

@vipannalla vipannalla Awaiting requested review from vipannalla

@jrplatin jrplatin Awaiting requested review from jrplatin jrplatin will be requested when the pull request is marked ready for review jrplatin is a code owner

@QiliangCui QiliangCui Awaiting requested review from QiliangCui QiliangCui will be requested when the pull request is marked ready for review QiliangCui is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ylangtsou @CienetStingLin @dennisYehCienet