Skip to content

Conversation

@Potabk
Copy link
Collaborator

@Potabk Potabk commented Jul 21, 2025

What this PR does / why we need it?

Fix #21227 to make ci happy

Does this PR introduce any user-facing change?

How was this patch tested?

Potabk added 3 commits July 21, 2025 22:06
Signed-off-by: wangli <wangli858794774@gmail.com>
Signed-off-by: wangli <wangli858794774@gmail.com>
Signed-off-by: wangli <wangli858794774@gmail.com>
@codecov
Copy link

codecov bot commented Jul 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.25%. Comparing base (7265dc0) to head (2b6f785).
⚠️ Report is 545 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1915   +/-   ##
=======================================
  Coverage   60.25%   60.25%           
=======================================
  Files          73       73           
  Lines        8006     8006           
=======================================
  Hits         4824     4824           
  Misses       3182     3182           
Flag Coverage Δ
unittests 60.25% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wangxiyuan wangxiyuan merged commit 33e1ea4 into vllm-project:main Jul 22, 2025
25 checks passed
@Potabk Potabk deleted the fix_pooler branch July 22, 2025 01:07
@Yikun Yikun changed the title [CI] Fix broken CI [CI] Fix broken CI by supporting multiple poolers at model level Sep 21, 2025
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Sep 26, 2025
### What this PR does / why we need it?
Fix [#21227](vllm-project/vllm#21227) to make ci
happy

- vLLM version: v0.9.2
- vLLM main:
vllm-project/vllm@6b46c4b

---------

Signed-off-by: wangli <wangli858794774@gmail.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
### What this PR does / why we need it?
Fix [#21227](vllm-project/vllm#21227) to make ci
happy

- vLLM version: v0.9.2
- vLLM main:
vllm-project/vllm@6b46c4b

---------

Signed-off-by: wangli <wangli858794774@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.

3 participants