Skip to content

Enable push trigger for image job#2906

Merged
wangxiyuan merged 1 commit intovllm-project:mainfrom
Yikun:main-img-build
Sep 13, 2025
Merged

Enable push trigger for image job#2906
wangxiyuan merged 1 commit intovllm-project:mainfrom
Yikun:main-img-build

Conversation

@Yikun
Copy link
Copy Markdown
Member

@Yikun Yikun commented Sep 13, 2025

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

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions
Copy link
Copy Markdown
Contributor

👋 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 ready-for-test start test by label for PR and removed ci/build labels Sep 13, 2025
@wangxiyuan wangxiyuan merged commit d2250c8 into vllm-project:main Sep 13, 2025
36 of 38 checks passed
@Yikun
Copy link
Copy Markdown
Member Author

Yikun commented Sep 13, 2025

image

Confirm this worked as expected.

offline893 pushed a commit to offline893/vllm-ascend that referenced this pull request Sep 16, 2025
### 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
### 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
### 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
### 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
### 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
### 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

Labels

ready read for review ready-for-test start test by label for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants