Skip to content

Enable label-based image test and use free runner to run lint#2864

Merged
wangxiyuan merged 1 commit intovllm-project:mainfrom
Yikun:main-0102
Sep 12, 2025
Merged

Enable label-based image test and use free runner to run lint#2864
wangxiyuan merged 1 commit intovllm-project:mainfrom
Yikun:main-0102

Conversation

@Yikun
Copy link
Member

@Yikun Yikun commented Sep 10, 2025 •
edited by github-actions bot
Loading

What this PR does / why we need it?

  • Enable label-based image test and use free runner to run lint
  • soft revert 26f388b

Does this PR introduce any user-facing change?

No

How was this patch tested?

@github-actions
Copy link

github-actions bot commented Sep 10, 2025

👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:‌‌

  • A PR should do only one thing, smaller PRs enable faster reviews.
  • Every PR should include unit tests and end-to-end tests ‌to ensure it works and is not broken by other future PRs.
  • Write the commit message by fulfilling the PR description to help reviewer and future developers understand.

If CI fails, you can run linting and testing checks locally according Contributing and Testing.

@Yikun Yikun added ready read for review accuracy-test enable all accuracy test for PR performance-test enable performance test for PR ready-for-test start test by label for PR labels Sep 10, 2025
@Yikun Yikun force-pushed the main-0102 branch 2 times, most recently from 7c1364d to 81784cf Compare September 11, 2025 11:47
@Yikun Yikun changed the title Test on releases/0.10.2 Enable label-based image test and use free runner to run lint Sep 11, 2025
@codecov
Copy link

codecov bot commented Sep 11, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.06%. Comparing base (1bbb20e) to head (391b599).
⚠️ Report is 746 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2864      +/-   ##
==========================================
+ Coverage   74.76%   75.06%   +0.29%     
==========================================
  Files         150      154       +4     
  Lines       20891    21278     +387     
==========================================
+ Hits        15620    15972     +352     
- Misses       5271     5306      +35
Flag Coverage Δ
unittests 75.06% <ø> (+0.29%) ⬆️

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.

@Yikun Yikun added ready-for-test start test by label for PR and removed ready-for-test start test by label for PR labels Sep 11, 2025
@Yikun Yikun added ready-for-test start test by label for PR and removed ready-for-test start test by label for PR labels Sep 11, 2025
@Yikun
Enable label-based image test and free lint
391b599 
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
@wangxiyuan wangxiyuan marked this pull request as ready for review September 12, 2025 02:49
@wangxiyuan wangxiyuan merged commit 6d8bc38 into vllm-project:main Sep 12, 2025
13 checks passed
@Yikun Yikun mentioned this pull request Sep 13, 2025
Merged
wangxiyuan pushed a commit that referenced this pull request Sep 13, 2025
@Yikun
Enable push trigger for image job (#2906)
d2250c8 
### What this PR does / why we need it?
Enable push trigger for image job

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Followup on #2864
- vLLM version: v0.10.2rc2
- vLLM main:
vllm-project/vllm@89e08d6

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
offline893 pushed a commit to offline893/vllm-ascend that referenced this pull request Sep 16, 2025
@Yikun
Enable label-based image test and use free runner to run lint (vllm-p…
133d8ae 
…roject#2864)

### What this PR does / why we need it?
- Enable label-based image test and use free runner to run lint
- soft revert
vllm-project@26f388b

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?

- vLLM version: main
- vLLM main:
vllm-project/vllm@404c85c

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Signed-off-by: offline0806 <z00858301@china.huawei.com>
offline893 pushed a commit to offline893/vllm-ascend that referenced this pull request Sep 16, 2025
@Yikun
Enable push trigger for image job (vllm-project#2906)
d0a4e1b 
### What this PR does / why we need it?
Enable push trigger for image job

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Followup on vllm-project#2864
- vLLM version: v0.10.2rc2
- vLLM main:
vllm-project/vllm@89e08d6

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Signed-off-by: offline0806 <z00858301@china.huawei.com>
wangxiaoteng888 pushed a commit to LCAIZJ/vllm-ascend that referenced this pull request Sep 25, 2025
@Yikun @wangxiaoteng888
Enable label-based image test and use free runner to run lint (vllm-p…
4496e96 
…roject#2864)

### What this PR does / why we need it?
- Enable label-based image test and use free runner to run lint
- soft revert
vllm-project@26f388b

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?


- vLLM version: main
- vLLM main:
vllm-project/vllm@404c85c

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
wangxiaoteng888 pushed a commit to LCAIZJ/vllm-ascend that referenced this pull request Sep 25, 2025
@Yikun @wangxiaoteng888
Enable push trigger for image job (vllm-project#2906)
367f37d 
### What this PR does / why we need it?
Enable push trigger for image job

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Followup on vllm-project#2864
- vLLM version: v0.10.2rc2
- vLLM main:
vllm-project/vllm@89e08d6

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Sep 26, 2025
@Yikun
Enable label-based image test and use free runner to run lint (vllm-p…
e1cea65 
…roject#2864)

### What this PR does / why we need it?
- Enable label-based image test and use free runner to run lint
- soft revert
vllm-project@26f388b

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?


- vLLM version: main
- vLLM main:
vllm-project/vllm@404c85c

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Sep 26, 2025
@Yikun
Enable push trigger for image job (vllm-project#2906)
9baa542 
### What this PR does / why we need it?
Enable push trigger for image job

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Followup on vllm-project#2864
- vLLM version: v0.10.2rc2
- vLLM main:
vllm-project/vllm@89e08d6

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
@Yikun @Angazenn
Enable label-based image test and use free runner to run lint (vllm-p…
d5e48d1 
…roject#2864)

### What this PR does / why we need it?
- Enable label-based image test and use free runner to run lint
- soft revert
vllm-project@26f388b

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?


- vLLM version: main
- vLLM main:
vllm-project/vllm@404c85c

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
@Yikun @Angazenn
Enable push trigger for image job (vllm-project#2906)
02f4208 
### What this PR does / why we need it?
Enable push trigger for image job

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Followup on vllm-project#2864
- vLLM version: v0.10.2rc2
- vLLM main:
vllm-project/vllm@89e08d6

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
NSDie pushed a commit to NSDie/vllm-ascend that referenced this pull request Nov 24, 2025
@Yikun @NSDie
Enable label-based image test and use free runner to run lint (vllm-p…
0be32fe 
…roject#2864)

### What this PR does / why we need it?
- Enable label-based image test and use free runner to run lint
- soft revert
vllm-project@26f388b

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?

- vLLM version: main
- vLLM main:
vllm-project/vllm@404c85c

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Signed-off-by: nsdie <yeyifan@huawei.com>
NSDie pushed a commit to NSDie/vllm-ascend that referenced this pull request Nov 24, 2025
@Yikun @NSDie
Enable push trigger for image job (vllm-project#2906)
e772c1a 
### What this PR does / why we need it?
Enable push trigger for image job

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Followup on vllm-project#2864
- vLLM version: v0.10.2rc2
- vLLM main:
vllm-project/vllm@89e08d6

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Signed-off-by: nsdie <yeyifan@huawei.com>
Clorist33 pushed a commit to Clorist33/vllm-ascend that referenced this pull request Dec 9, 2025
@Yikun
Enable label-based image test and use free runner to run lint (vllm-p…
87aa75e 
…roject#2864)

### What this PR does / why we need it?
- Enable label-based image test and use free runner to run lint
- soft revert
vllm-project@26f388b

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?


- vLLM version: main
- vLLM main:
vllm-project/vllm@404c85c

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Clorist33 pushed a commit to Clorist33/vllm-ascend that referenced this pull request Dec 9, 2025
@Yikun
Enable push trigger for image job (vllm-project#2906)
d95e1d6 
### What this PR does / why we need it?
Enable push trigger for image job

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Followup on vllm-project#2864
- vLLM version: v0.10.2rc2
- vLLM main:
vllm-project/vllm@89e08d6

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wangxiyuan wangxiyuan wangxiyuan approved these changes

Assignees

No one assigned

Labels

accuracy-test enable all accuracy test for PR ci/build performance-test enable performance test for PR ready read for review ready-for-test start test by label for PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Yikun @wangxiyuan