Skip to content

Pass ROCm package index URL through JAX release workflow#65

Open
erman-gurses wants to merge 6 commits into
mainfrom
users/erman-gurses/update-jax
Open

Pass ROCm package index URL through JAX release workflow#65
erman-gurses wants to merge 6 commits into
mainfrom
users/erman-gurses/update-jax

Conversation

@erman-gurses

@erman-gurses erman-gurses commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Related PR: ROCm/TheRock#6054

Add rocm_package_find_links_url to the rockrel JAX release workflow and forward it to the reusable TheRock JAX build workflow.

This allows the JAX manylinux build path to consume ROCm packages from a configurable package index instead of relying on a hardcoded URL.

Changes

  • Add rocm_package_find_links_url workflow input to multi_arch_release_linux_jax_wheels.yml

  • Forward rocm_package_find_links_url to the reusable multi_arch_build_linux_jax_wheels.yml workflow

  • Default the value to the current dev multi-arch package index:

    • https://rocm.devreleases.amd.com/whl-multi-arch/

Motivation

The JAX manylinux build flow now installs ROCm packages from a package index. Passing the package source as an input allows the workflow to support different release channels (dev, nightly, prerelease) without hardcoding a specific index URL in the build workflow.

Tests:
https://github.com/ROCm/rockrel/actions/runs/28086158341

Comment on lines +21 to +24
rocm_package_find_links_url:
description: "ROCm package index / find-links URL for the JAX manylinux build."
type: string
default: "https://rocm.devreleases.amd.com/whl-multi-arch/"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned at ROCm/TheRock#6054 (comment), this default URL is not compatible with --find-links. Should be clear on what index type the workflows need.

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.

2 participants