Skip to content

[release/7.11] [CI] Adjust Linux release for rockrel#3055

Merged
HereThereBeDragons merged 4 commits into
release/therock-7.11from
users/lpromber/7.11_workflow_adjust
Jan 23, 2026
Merged

[release/7.11] [CI] Adjust Linux release for rockrel#3055
HereThereBeDragons merged 4 commits into
release/therock-7.11from
users/lpromber/7.11_workflow_adjust

Conversation

@HereThereBeDragons
Copy link
Copy Markdown
Contributor

@HereThereBeDragons HereThereBeDragons commented Jan 22, 2026

Adjust to make .github/workflows/release_portable_linux_packages.yml propagate all needed params to native packaging, so that it can be triggered from outside TheRock.

Comment thread .github/workflows/release_portable_linux_packages.yml Outdated
Comment thread .github/workflows/release_portable_linux_packages.yml Outdated
Copy link
Copy Markdown
Member

@marbre marbre left a comment

Choose a reason for hiding this comment

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

The changes to .github/workflows/build_native_linux_packages.yml need to go to main as well. I'd rather split it therefore to be able to apply the same patch to main as well as the release branch. JAX can be an additional, separate patch.

Comment thread .github/workflows/build_native_linux_packages.yml Outdated
Comment thread .github/workflows/build_native_linux_packages.yml
@HereThereBeDragons HereThereBeDragons changed the title [release/7.11] [CI] Adjust Linux release for rockrel, remove jax [release/7.11] [CI] Adjust Linux release for rockrel Jan 23, 2026
@HereThereBeDragons
Copy link
Copy Markdown
Contributor Author

removed the part about jax

Comment thread .github/workflows/build_native_linux_packages.yml Outdated
python ./build_tools/fetch_artifacts.py \
--run-id=${{ env.ARTIFACT_RUN_ID }} \
--run-github-repo="ROCm/TheRock" \
--run-github-repo=${{ inputs.repository || 'ROCm/TheRock' }} \
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.

Not sure about the mechanism behind when passing --run-github-repo. This probably needs a careful look.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i will put it back to the hardcoded rocm/therock.
@ScottTodd probably knows better if something else is safe.

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.

I don't think this is going to work when glancing at

.

Copy link
Copy Markdown
Member

@marbre marbre left a comment

Choose a reason for hiding this comment

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

Workflow adjustment look correct but installing artifacts won't work as far as I see.

@HereThereBeDragons HereThereBeDragons merged commit dedfe80 into release/therock-7.11 Jan 23, 2026
5 checks passed
@HereThereBeDragons HereThereBeDragons deleted the users/lpromber/7.11_workflow_adjust branch January 23, 2026 13:35
@github-project-automation github-project-automation Bot moved this from TODO to Done in TheRock Triage Jan 23, 2026
@HereThereBeDragons
Copy link
Copy Markdown
Contributor Author

if it doesnt work then we need to have a closer look to what .github/workflows/build_portable_linux_python_packages.yml does

nunnikri pushed a commit that referenced this pull request Feb 20, 2026
Adjust to make `.github/workflows/release_portable_linux_packages.yml`
propagate all needed params to native packaging, so that it can be
triggered from outside TheRock.
nunnikri added a commit that referenced this pull request Feb 20, 2026
## Motivation

Cherry pick #3055 and #3084 

## Technical Details

Cherry picking changes directly landed in 7.11 branch to main

---------

Co-authored-by: Laura Promberger <laura.promberger@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants