Skip to content

Clean up obsoleted vLLM tests#163383

Closed
huydhn wants to merge 1 commit intopytorch:mainfrom
huydhn:remove-vllm-obsoleted-tests
Closed

Clean up obsoleted vLLM tests#163383
huydhn wants to merge 1 commit intopytorch:mainfrom
huydhn:remove-vllm-obsoleted-tests

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Sep 19, 2025

They have been removed in vllm-project/vllm#25117 and vllm-project/vllm#22772, thus failing in trunk at the moment after the latest pin commit update

cc @zou3519

Signed-off-by: Huy Do <huydhn@gmail.com>
@huydhn huydhn requested a review from zou3519 September 19, 2025 23:11
@huydhn huydhn requested a review from a team as a code owner September 19, 2025 23:11
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 19, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/163383

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit af152f2 with merge base 33e6c5a (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@huydhn
Copy link
Contributor Author

huydhn commented Sep 20, 2025

@pytorchbot merge -f 'vLLM tests have passed'

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

mansiag05 pushed a commit to mansiag05/pytorch that referenced this pull request Sep 22, 2025
They have been removed in vllm-project/vllm#25117 and vllm-project/vllm#22772, thus failing in trunk at the moment after the latest pin commit update

Pull Request resolved: pytorch#163383
Approved by: https://github.com/wdvr, https://github.com/seemethere, https://github.com/malfet
cleonard530 pushed a commit to cleonard530/pytorch that referenced this pull request Sep 22, 2025
They have been removed in vllm-project/vllm#25117 and vllm-project/vllm#22772, thus failing in trunk at the moment after the latest pin commit update

Pull Request resolved: pytorch#163383
Approved by: https://github.com/wdvr, https://github.com/seemethere, https://github.com/malfet
dsashidh pushed a commit to dsashidh/pytorch that referenced this pull request Sep 26, 2025
They have been removed in vllm-project/vllm#25117 and vllm-project/vllm#22772, thus failing in trunk at the moment after the latest pin commit update

Pull Request resolved: pytorch#163383
Approved by: https://github.com/wdvr, https://github.com/seemethere, https://github.com/malfet
@Camyll
Copy link
Contributor

Camyll commented Sep 30, 2025

@pytorchbot cherry-pick --onto release/2.9 -c critical

pytorchbot pushed a commit that referenced this pull request Sep 30, 2025
They have been removed in vllm-project/vllm#25117 and vllm-project/vllm#22772, thus failing in trunk at the moment after the latest pin commit update

Pull Request resolved: #163383
Approved by: https://github.com/wdvr, https://github.com/seemethere, https://github.com/malfet

(cherry picked from commit a31acf3)
@pytorchbot
Copy link
Collaborator

Cherry picking #163383

The cherry pick PR is at #164282 and it is recommended to link a critical cherry pick PR with an issue. The following tracker issues are updated:

Details for Dev Infra team Raised by workflow job

Camyll pushed a commit that referenced this pull request Sep 30, 2025
Clean up obsoleted vLLM tests (#163383)

They have been removed in vllm-project/vllm#25117 and vllm-project/vllm#22772, thus failing in trunk at the moment after the latest pin commit update

Pull Request resolved: #163383
Approved by: https://github.com/wdvr, https://github.com/seemethere, https://github.com/malfet

(cherry picked from commit a31acf3)

Co-authored-by: Huy Do <huydhn@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants