Skip to content

[CI] Unpin exact triton wheel hash to fix install failure#600

Open
coderfeli wants to merge 1 commit into
mainfrom
fix/ci-unpin-triton-wheel
Open

[CI] Unpin exact triton wheel hash to fix install failure#600
coderfeli wants to merge 1 commit into
mainfrom
fix/ci-unpin-triton-wheel

Conversation

@coderfeli
Copy link
Copy Markdown
Collaborator

Problem

CI installs triton with an exact build-hash pin:

triton==3.7.0+amd.rocm7.2.0.gitd1660454

AMD overwrites the .gitXXXXXXX build-hash suffix whenever they publish a new wheel for a given ROCm version (gitd1660454gitd0d77a509). When that happens the pinned wheel vanishes from the index, so pip install fails and the test job dies before running anything. This pin predates #597 — it's repo-history legacy living on main, so every branch inherits the breakage.

Fix

Install triton unpinned from the same ROCm 7.2.0 index, and let the already-present >=3.6.0 assertion on the next line act as the version floor:

- ... --extra-index-url https://pypi.amd.com/triton/rocm-7.2.0/simple/ 'triton==3.7.0+amd.rocm7.2.0.gitd1660454'"
+ ... --extra-index-url https://pypi.amd.com/triton/rocm-7.2.0/simple/ triton"

This mirrors aiter's install_triton.sh, which never pins the volatile build hash — it installs triton from the index and verifies >=3.6.0.

Scope

One line in .github/workflows/flydsl.yaml. Fixes CI repo-wide; other open branches (e.g. #597) pick it up on their next rebase onto main.

🤖 Generated with Claude Code

The CI pinned `triton==3.7.0+amd.rocm7.2.0.gitd1660454`, but AMD overwrites
the build-hash suffix whenever they publish a new wheel for a given ROCm
version (gitd1660454 -> gitd0d77a509), so the pinned wheel disappears from
the index and `pip install` fails before tests can run.

Install `triton` unpinned from the same ROCm 7.2.0 index and rely on the
existing `>=3.6.0` assertion (next line) as the version floor, mirroring
aiter's install_triton.sh, which never pins the volatile `.gitXXXX` hash.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant