[CI] test e2e - #257
Open
zhangxinyuehfad wants to merge 20 commits into
Open
Conversation
Signed-off-by: wjunLu <wjunlu217@gmail.com>
Signed-off-by: MrZ20 <2609716663@qq.com>
Contributor
|
👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:
If CI fails, you can run linting and testing checks locally according Contributing and Testing. |
Collaborator
Author
|
/e2e tests/e2e/pull_request/four_card/test_qwen3_6.py |
Collaborator
Author
|
/e2e tests/e2e/pull_request/four_card/test_qwen3_6.py --vllm 9090368b650896bf5fc990c921df7eb4c20355a5, v0.21.0 |
Collaborator
Author
|
/e2e tests/e2e/pull_request/four_card/test_qwen3_6.py --vllm 9090368b650896bf5fc990c921df7eb4c20355a5,v0.21.0 |
Signed-off-by: hfadzxy <starmoon_zhang@163.com>
Collaborator
Author
|
/e2e tests/e2e/pull_request/four_card/test_qwen3_6.py --vllm 9090368b650896bf5fc990c921df7eb4c20355a5,v0.21.0 |
### What this PR does / why we need it? Fixes #9800. This PR adds two repository-local verified vLLM refs as the shared source of truth for PR selected tests and docs metadata: - `.github/vllm-main-verified.commit`: verified upstream vLLM main commit. - `.github/vllm-release-tag.commit`: matched vLLM release tag. The refs stay in separate files because they have different semantics. The workflow wiring keeps the variable flow explicit and small: - `lint-and-select-tests` reads both files in one step after checkout. - `main_commit` is validated as a 7-40 character hex commit and used for the vLLM checkout plus selected-test matrix. - `release_tag` is validated as a vLLM release tag and used as the second selected-test matrix entry. - `/e2e` command selected tests in `.github/workflows/pr_e2e_command.yml` use the same `main_commit` / `release_tag` outputs, so they no longer keep a separate hardcoded `vllm_version` list. - `docs/source/conf.py` directly reads the same anchor files for `main_vllm_commit` and `main_vllm_tag`. This intentionally avoids a separate helper job. Dockerfile defaults and scheduled workflow hardcode cleanup are split into follow-up PRs. No. This is CI/tooling and docs metadata plumbing only. Local checks: - `python docs/source/conf.py` - `python -m ruff check docs/source/conf.py` - `python -m ruff format --check docs/source/conf.py` - `actionlint .github/workflows/pr_test.yaml .github/workflows/pr_e2e_command.yml` - `git diff --check` Conflict resolution: - Rebased onto latest `vllm-project:main`. - Preserved the latest `lint-and-select-tests` source-filter flow. - Added the missing `trigger-selected-tests` `vllm_version` wiring in `.github/workflows/pr_e2e_command.yml`. - vLLM version: v0.21.0 - vLLM main: vllm-project/vllm@9090368 Signed-off-by: robellliu-dev <robell.liu@huawei.com> Signed-off-by: hfadzxy <starmoon_zhang@163.com>
Signed-off-by: hfadzxy <starmoon_zhang@163.com>
zhangxinyuehfad
force-pushed
the
zxy_e2e_test
branch
from
June 10, 2026 09:39
7a3bf34 to
71dbab8
Compare
zhangxinyuehfad
force-pushed
the
main
branch
2 times, most recently
from
June 18, 2026 04:04
de24f70 to
60cd7c9
Compare
zhangxinyuehfad
force-pushed
the
main
branch
11 times, most recently
from
June 26, 2026 09:09
05b98c1 to
38085d5
Compare
zhangxinyuehfad
force-pushed
the
main
branch
6 times, most recently
from
July 7, 2026 03:28
1cd9ff4 to
b7570c6
Compare
zhangxinyuehfad
force-pushed
the
main
branch
4 times, most recently
from
July 13, 2026 09:44
0c96baa to
e7d055b
Compare
MrZ20
force-pushed
the
main
branch
3 times, most recently
from
July 24, 2026 12:44
8899462 to
ce1086a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it?
Does this PR introduce any user-facing change?
How was this patch tested?