[ROCm] [Bugfix] [Release] Fix nightly rocm release pipeline

[tjtanaa]

Purpose

Fix https://buildkite.com/vllm/release-v2/builds/92/steps/canvas?sid=019d2b00-5a75-412f-a8f2-9607ef7e0ddc&tab=output /bin/bash: line 57: ECR_IMAGE_TAG: unbound variable introduced by #37283

Test Plan

Test Result

---

Essential Elements of an Effective PR Description Checklist

• The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
• The test plan, such as providing test command.
• The test results, such as pasting the results comparison before and after, or e2e results
• (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
• (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

[gemini-code-assist]

Code Review

This pull request updates the Buildkite release pipeline by commenting out CUDA and CPU build steps and redirecting ROCm artifacts to a development S3 bucket. The ROCm pipeline is further modified with updated caching logic and variable escaping; however, the reviewer identified that caching is currently broken due to the removal of the push flag and a hardcoded variable override that forces a cache miss.

[tjtanaa]

This case (scenario 2) is not useful as we are not pushing all of the intermediate stages to the ECR:

• base
• build_triton
• build_pytorch
• build_fa
• build_aiter
• build_amdsmi
• build_mori
• debs
• debs_wheel_release

So we are not able to reuse the cache from base image through --cache-from type=registry,ref=public.ecr.aws/q9t5s3a7/vllm-release-repo:1227c9527d573e09-rocm-base

Example command:

DOCKER_BUILDKIT=1 docker buildx build \
  --file docker/Dockerfile.rocm_base \
  --tag rocm-base-debs:$${BUILDKITE_BUILD_NUMBER} \
  --target debs_wheel_release \
  --cache-from type=registry,ref=public.ecr.aws/q9t5s3a7/vllm-release-repo:1227c9527d573e09-rocm-base \
  --build-arg USE_SCCACHE=1 \
  --build-arg SCCACHE_BUCKET_NAME=vllm-build-sccache \
  --build-arg SCCACHE_REGION_NAME=us-west-2 \
  --build-arg SCCACHE_S3_NO_CREDENTIALS=0 \
  --load \
  .

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[tjtanaa]

@claude review

[tjtanaa]

@gemini review

[gemini-code-assist]

Code Review

This pull request removes the partial cache handling logic in the ROCm release pipeline and corrects a variable interpolation error for the build number. The review feedback points out that removing the partial cache logic may cause performance regressions and suggests a specific fix to resolve the underlying issue while retaining the cache optimization.