Skip to content

GH-90: Add script and workflow for updating the XLA commit hash#6

Merged
mrodden merged 1 commit intomasterfrom
gh90-xla-sync
May 15, 2025
Merged

GH-90: Add script and workflow for updating the XLA commit hash#6
mrodden merged 1 commit intomasterfrom
gh90-xla-sync

Conversation

@charleshofer
Copy link
Collaborator

Adds a workflow that lets us update the XLA commit hash on a given branch. This is mostly a port from the script in the old rocm/jax repository: ROCm/jax#330

Story: https://github.com/ROCm/jax-internal/issues/90

@charleshofer charleshofer changed the title Add script and workflow for updating the XLA commit hash GH-90: Add script and workflow for updating the XLA commit hash May 14, 2025
Copy link
Contributor

@mrodden mrodden left a comment

Choose a reason for hiding this comment

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

One change request inline

@@ -0,0 +1,120 @@
#!/usr/bin/env python3
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we move this to tools/ instead? I was gonna ask you to port this over earlier because I was hoping to use it when bumping XLA and the plugin to version 0.7.0

app-id: ${{ vars.ROCM_REPO_MANAGEMENT_API_2_ID }}
private-key: ${{ secrets.ROCM_REPO_MANAGEMENT_API_2_PRIV_KEY }}
- name: Run update script
run: python3 ci/update_xla_hash.py -v --xla-repo ${{ inputs.xla_repo }} --gh-token $GH_TOKEN ${{ inputs.xla_hash }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Gotta fix the path here now

Suggested change
run: python3 ci/update_xla_hash.py -v --xla-repo ${{ inputs.xla_repo }} --gh-token $GH_TOKEN ${{ inputs.xla_hash }}
run: python3 tools/update_xla_hash.py -v --xla-repo ${{ inputs.xla_repo }} --gh-token $GH_TOKEN ${{ inputs.xla_hash }}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

Adds a workflow that lets us update the XLA commit hash on a given
branch. This is mostly a port from the script in the old rocm/jax
repository: ROCm/jax#330
@mrodden mrodden merged commit bb43aa2 into master May 15, 2025
@mrodden mrodden deleted the gh90-xla-sync branch May 15, 2025 16:10
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.

3 participants