执行摘要
本PR更新ROCm变体至7.2.1,调整CI/CD发布流水线配置,确保版本一致性,是PR #38252的后续工作,风险较低,主要涉及基础设施维护。
功能与动机
跟随PR #38252,将ROCm版本从7.0.0升级到7.2.1,以保持发布流程的最新性和兼容性。PR body中明确提到此PR是#38252的后续,目的是更新ROCm变体。
实现拆解
主要修改两个文件:
.buildkite/release-pipeline.yaml:将VARIANT从rocm700改为rocm721,并添加if条件if: build.env("NIGHTLY") == "1"限制nightly镜像发布。
tools/vllm-rocm/generate-rocm-wheels-root-index.sh:更新默认VARIANT值从rocm700到rocm721。
这些变更确保CI/CD流水线使用正确的ROCm变体,避免配置不一致。
评论区精华
review中,gemini-code-assist[bot]指出一个关键问题:
"This inconsistency is critical. It will cause wheels built with ROCm 7.0 to be published under a path that indicates they are for ROCm 7.2.1. This will likely lead to incorrect dependencies and runtime failures for users."
作者tjtanaa回复:
"This is the case before I synced to main. I have rebased, now it is pointing to rocm 7.2.1"
讨论焦点是版本一致性的正确性风险,结论是问题在同步main分支后已解决。
风险与影响
风险:如果版本不一致未修复,可能导致发布错误版本的wheel,用户使用时出现依赖错误或运行时失败。但作者确认已修复,风险较低。
影响:
- 对系统:更新CI/CD发布流程,确保使用ROCm 7.2.1版本。
- 对用户:修复后无直接影响;未修复可能导致兼容性问题。
影响范围限于ROCm相关构建和发布,不涉及核心模型逻辑。
关联脉络
此PR与PR #38252直接关联,后者更新了Docker基础镜像至ROCm 7.2.1。结合近期历史PR如#38367(文档更新)和#37453(ROCm bugfix),可见vllm仓库正持续优化ROCm支持,体现了基础设施版本管理的演进趋势。
参与讨论