Skip to content

Rename project_to_test to projects_to_test and update TheRock (2026-02-02)#3020

Merged
jayhawk-commits merged 2 commits intodevelopfrom
users/jayhawk-commits/therock-sha-feb2
Feb 3, 2026
Merged

Rename project_to_test to projects_to_test and update TheRock (2026-02-02)#3020
jayhawk-commits merged 2 commits intodevelopfrom
users/jayhawk-commits/therock-sha-feb2

Conversation

@jayhawk-commits
Copy link
Collaborator

  • Also update runner label from linux-mi325-1gpu-ossci-rocm-frac to linux-mi325-1gpu-ossci-rocm

…2026-02-02

Also update runner label from linux-mi325-1gpu-ossci-rocm-frac to linux-mi325-1gpu-ossci-rocm
@jayhawk-commits jayhawk-commits requested review from a team and jrmadsen as code owners February 2, 2026 19:01
Copilot AI review requested due to automatic review settings February 2, 2026 19:01
@jayhawk-commits jayhawk-commits requested review from a team as code owners February 2, 2026 19:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR renames the parameter project_to_test to projects_to_test across TheRock CI workflows and updates TheRock references to a newer commit (2026-02-02). The runner label for MI325 GPU tests is also updated from linux-mi325-1gpu-ossci-rocm-frac to linux-mi325-1gpu-ossci-rocm.

Changes:

  • Renamed project_to_test to projects_to_test across all workflow files and Python scripts
  • Updated TheRock repository commit references from older dates to commit 846614189049e22b02c85bf2b496a9025b27e75d (2026-02-02)
  • Updated MI325 runner labels from -frac suffix to standard naming convention

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
projects/rocprofiler-systems/.github/ci-matrix.yml Updated MI325 runner labels to remove -frac suffix
projects/rocprofiler-compute/.github/ci-matrix.yml Updated MI325 runner labels to remove -frac suffix
projects/rccl/.github/workflows/therock-test-packages-single-node.yml Updated TheRock commit reference to 2026-02-02
projects/rccl/.github/workflows/therock-test-packages-multi-node.yml Updated TheRock commit reference to 2026-02-02
projects/rccl/.github/workflows/therock-ci-linux.yml Updated TheRock commit reference to 2026-02-02
.github/workflows/therock-test-packages.yml Renamed project_to_test to projects_to_test and updated TheRock references
.github/workflows/therock-ci.yml Renamed project_to_test to projects_to_test in job names and parameters
.github/workflows/therock-ci-windows.yml Renamed project_to_test to projects_to_test and updated TheRock reference
.github/workflows/therock-ci-linux.yml Renamed project_to_test to projects_to_test, updated TheRock reference and runner label
.github/workflows/rocprofiler-sdk-docs.yml Updated MI325 runner label to remove -frac suffix
.github/workflows/rocprofiler-sdk-continuous_integration.yml Updated MI325 runner labels to remove -frac suffix
.github/workflows/rocprofiler-register-continuous-integration.yml Updated MI325 runner label to remove -frac suffix
.github/workflows/aqlprofile-continuous_integration.yml Updated MI325 runner labels to remove -frac suffix
.github/scripts/therock_matrix.py Renamed project_to_test to projects_to_test and reformatted quotes from single to double

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@geomin12 geomin12 left a comment

Choose a reason for hiding this comment

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

lgtm assuming CI passes

Copy link
Collaborator

@jbonnell-amd jbonnell-amd left a comment

Choose a reason for hiding this comment

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

Thanks for updating the runner names for the profiler workflow/matrix files! LGTM

@jayhawk-commits
Copy link
Collaborator Author

Hitting Update Branch to quickly re-trigger all CI, due to github actions outage earlier.

@jayhawk-commits
Copy link
Collaborator Author

CI failures do not look related to the changes being made here.

@jayhawk-commits jayhawk-commits merged commit 98d6ca1 into develop Feb 3, 2026
86 of 99 checks passed
@jayhawk-commits jayhawk-commits deleted the users/jayhawk-commits/therock-sha-feb2 branch February 3, 2026 15:56
systems-assistant bot pushed a commit to ROCm/rocprofiler-compute that referenced this pull request Feb 3, 2026
 SHA to 2026-02-02 (#3020)

- Also update runner label from linux-mi325-1gpu-ossci-rocm-frac to linux-mi325-1gpu-ossci-rocm
[rocm-systems] ROCm/rocm-systems#3020 (commit 98d6ca1)
systems-assistant bot pushed a commit to ROCm/rocprofiler-systems that referenced this pull request Feb 3, 2026
 SHA to 2026-02-02 (#3020)

- Also update runner label from linux-mi325-1gpu-ossci-rocm-frac to linux-mi325-1gpu-ossci-rocm
[rocm-systems] ROCm/rocm-systems#3020 (commit 98d6ca1)
systems-assistant bot pushed a commit to ROCm/rccl that referenced this pull request Feb 3, 2026
 SHA to 2026-02-02 (#3020)

- Also update runner label from linux-mi325-1gpu-ossci-rocm-frac to linux-mi325-1gpu-ossci-rocm
[rocm-systems] ROCm/rocm-systems#3020 (commit 98d6ca1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants