Skip to content

[bkc] Pin to use release/bkc/therock-10.2-20261013 workflows - #13560

Merged
ScottTodd merged 2 commits into
release/bkc/therock-10.2-20261013from
users/scotttodd/bkc-10.2-020261013-pins
Oct 14, 2026
Merged

ScottTodd merged 2 commits into
release/bkc/therock-10.2-20261013from
users/scotttodd/bkc-10.2-020261013-pins

Conversation

@ScottTodd

@ScottTodd ScottTodd commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Motivation

The next BKC release branch was cut and this pins the workflows to use code from it.

Technical Details

Submission Checklist

@github-actions github-actions Bot added github actions project: none Does not target any component labels Oct 13, 2026
@ScottTodd

Copy link
Copy Markdown
Member Author

CI is failing on setup here, needs #13525 also. cc @geomin12

@geomin12

Copy link
Copy Markdown
Collaborator

CI is failing on setup here, needs #13525 also. cc @geomin12

FYI i have landed #13525

geomin12 and others added 2 commits October 14, 2026 08:19
…d skip-ci-config.toml (#13525)

Bumps all pinned TheRock workflow and source refs to `c01e3a3`, which
adds path-based skip filtering for external repos (ROCm/TheRock#8800).

That feature reads skip patterns from a TOML config sourced from the
external repo, so this also adds `.github/skip-ci-config.toml` (seeded
with the existing `SKIPPABLE_PATH_PATTERNS` from
`.github/scripts/therock_configure_ci.py`) and passes the SHAs, event
metadata, and config path through the `external_repo` payload so TheRock
can compute the diff and decide whether CI is needed.

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
(cherry picked from commit fad26ec)
Use release/bkc/therock-10.2-20261013 for reusable workflows, checkout refs, the CI environment output, and the ref resolver default. Preserve workflow overrides and leave artifact reuse policy in TheRock.

Validation: pre-commit and actionlint passed; 11 ref resolver tests passed.

Generated with Codex
@ScottTodd
ScottTodd force-pushed the users/scotttodd/bkc-10.2-020261013-pins branch from 48d2e62 to eca369a Compare October 14, 2026 15:28
@ScottTodd
ScottTodd marked this pull request as ready for review October 14, 2026 15:34
@ScottTodd
ScottTodd requested review from a team October 14, 2026 15:34
@ScottTodd
ScottTodd requested a review from a team as a code owner October 14, 2026 15:34
@ScottTodd

Copy link
Copy Markdown
Member Author

CI is failing on setup here, needs #13525 also. cc @geo

CI is failing on setup here, needs #13525 also. cc @geomin12

FYI i have landed #13525

Thanks. Cherry-picked that and rebased, resolving merge conflicts.

@araravik-psd araravik-psd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, see that the main changes are remnants from the first cherry pick and the other pinning part seems to be accurate, verified with files from older pinning commit for bkc releases https://github.com/ROCm/rocm-libraries/pull/12431/files#diff-8eb6d5f23517edac4c4d5cc5ff533efda680aab18c13300e3a0107bd709f02a5

baseline-run-id:
description: "TheRock workflow run ID for prebuilt stage artifacts"
value: "37655107204" # Updated by assistant-librarian[bot] on submodule bumps
value: "38220501023" # Updated by assistant-librarian[bot] on submodule bumps

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is pointing to the artifacts from the rocm-libs submodule bump run, which I believe is expected ROCm/TheRock#8933

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Waiting on reviews for ROCm/TheRock#8984 then all baseline run id / stage reuse code will be disabled on release branches.

# event_name tells TheRock this is a schedule/workflow_dispatch run with no
# meaningful diff, so it always runs CI. Without it, TheRock falls back to
# `git diff HEAD^` against a depth-1 checkout, which fails.
external_repo: >-

@araravik-psd araravik-psd Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See these changes have already landed on develop and cherry picked

@ScottTodd

Copy link
Copy Markdown
Member Author

CI signal looks clean enough to merge so the branch can be ready to receive subproject cherry-pick requests.

@ScottTodd
ScottTodd merged commit 9bfdb23 into release/bkc/therock-10.2-20261013 Oct 14, 2026
136 checks passed
@ScottTodd
ScottTodd deleted the users/scotttodd/bkc-10.2-020261013-pins branch October 14, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants