Skip to content

[CI] Resolve release V2 docker build after ROCm CI wheels change - #44463

Merged
khluu merged 1 commit into
vllm-project:mainfrom
ROCm:akaratza_fix_release_v2_failure
Jun 4, 2026
Merged

[CI] Resolve release V2 docker build after ROCm CI wheels change#44463
khluu merged 1 commit into
vllm-project:mainfrom
ROCm:akaratza_fix_release_v2_failure

Conversation

@AndreasKaratzas

Copy link
Copy Markdown
Member

Release Docker builds were failing with Repo is dirty because tracked files excluded by .dockerignore were missing from the container worktree while .git was still available for the cleanliness check. This keeps .clang-format and .gitattributes in the Docker context, while continuing to ignore non-build paths such as docs/, .github/, .pre-commit-config.yaml, and format.sh. To make that safe, tools/check_repo.sh restores only those intentionally omitted tracked paths from the Git index before running git diff --quiet inside Docker. This preserves the smaller Docker context for CI base builds without making release builds see false dirty-tree changes. Verified with git diff --check, bash -n tools/check_repo.sh, and a temp worktree simulation with the ignored tracked paths removed.

Signed-off-by: Andreas Karatzas <akaratza@amd.com>
@AndreasKaratzas
AndreasKaratzas marked this pull request as ready for review June 3, 2026 23:48
@AndreasKaratzas
AndreasKaratzas requested a review from khluu June 3, 2026 23:48
@mergify mergify Bot added the rocm Related to AMD ROCm label Jun 3, 2026
@github-project-automation github-project-automation Bot moved this to Todo in AMD Jun 3, 2026
@AndreasKaratzas AndreasKaratzas added the ready ONLY add when PR is ready to merge/full CI is needed label Jun 3, 2026
@khluu
khluu merged commit 5e2af28 into vllm-project:main Jun 4, 2026
17 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in AMD Jun 4, 2026
@AndreasKaratzas
AndreasKaratzas deleted the akaratza_fix_release_v2_failure branch June 4, 2026 04:47
mvanhorn pushed a commit to mvanhorn/vllm that referenced this pull request Jun 4, 2026
…m-project#44463)

Signed-off-by: Andreas Karatzas <akaratza@amd.com>
Signed-off-by: Matt Van Horn <455140+mvanhorn@users.noreply.github.com>
JisoLya pushed a commit to JisoLya/vllm that referenced this pull request Jun 5, 2026
…m-project#44463)

Signed-off-by: Andreas Karatzas <akaratza@amd.com>
Signed-off-by: JisoLya <523420504@qq.com>
knight0528 pushed a commit to knight0528/vllm that referenced this pull request Jun 8, 2026
waqahmed-amd-fi pushed a commit to waqahmed-amd-fi/vllm that referenced this pull request Jun 10, 2026
…m-project#44463)

Signed-off-by: Andreas Karatzas <akaratza@amd.com>
Signed-off-by: Waqar Ahmed <waqar.ahmed@amd.com>
Jackie2049 pushed a commit to Jackie2049/vllm that referenced this pull request Jun 12, 2026
Saddss pushed a commit to Saddss/vllm that referenced this pull request Jun 14, 2026
divineearthly pushed a commit to divineearthly/vllm that referenced this pull request Jun 19, 2026
…m-project#44463)

Signed-off-by: Andreas Karatzas <akaratza@amd.com>
Signed-off-by: divineearthly <divineearthly@gmail.com>
nkzhenhua pushed a commit to nkzhenhua/vllm that referenced this pull request Jun 24, 2026
ohsono pushed a commit to ohsono/vllm that referenced this pull request Jul 3, 2026
philippesic pushed a commit to philippesic/vllm-semantic-cache that referenced this pull request Jul 19, 2026
plasticchris pushed a commit to plasticchris/vllm that referenced this pull request Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed rocm Related to AMD ROCm

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants