Skip to content

Bump CI to 7.2.2#4805

Open
causten wants to merge 11 commits intodevelopfrom
update7.2
Open

Bump CI to 7.2.2#4805
causten wants to merge 11 commits intodevelopfrom
update7.2

Conversation

@causten
Copy link
Copy Markdown
Collaborator

@causten causten commented Apr 20, 2026

Motivation

ROCm 7.2.2 has been released. Move CI to match

Technical Details

Changelog Category

Add a CHANGELOG.md entry for any option other than Not Applicable

    • Added: New functionality.
    • Changed: Changes to existing functionality.
    • Removed: Functionality or support that has been removed. (Compared to a previous release)
    • Optimized: Component performance that has been optimized or improved.
    • Resolved Issues: Known issues from a previous version that have been resolved.
    • Not Applicable: This PR is not to be included in the changelog.

Copilot AI review requested due to automatic review settings April 20, 2026 22:12
@causten causten requested a review from a team as a code owner April 20, 2026 22:12
@causten causten changed the title Bump CIO to 7.2.2 Bump CI to 7.2.2 Apr 20, 2026
Copy link
Copy Markdown
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

Updates the repository’s Docker build inputs and GitHub workflows to target the newly released ROCm 7.2.2.

Changes:

  • Bump ROCm repository references from 7.1.1 to 7.2.2 in Docker build files (Ubuntu + SLES).
  • Update PyTorch/torchvision/triton wheel URLs in the main Dockerfile to the ROCm 7.2.2 builds.
  • Update default ROCm version inputs in multiple GitHub Actions workflows/config.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tools/docker/sles.docker Updates SLES zypper ROCm repo baseurl to 7.2.2.
hip-clang.docker Updates Ubuntu ROCm apt repo URL to 7.2.2.
Dockerfile Updates Ubuntu ROCm apt repo URL and ROCm-specific PyTorch wheel URLs to 7.2.2.
.github/workflows/unreleased_rocm.yaml Updates default rocm_release input to 7.2.2.
.github/workflows/rocm-image-release.yaml Updates fallback rocm_release to 7.2.2.
.github/workflows/performance.yaml Updates default rocm_release input to 7.2.2.
.github/workflows/config.md Updates documented ROCM_VERSION to 7.2.2.
.github/workflows/benchmark.yaml Updates rocm_version to 7.2.2.

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

Comment thread Dockerfile Outdated
Comment thread hip-clang.docker Outdated
Comment thread .github/workflows/config.md
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

❌ Patch coverage is 82.89474% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/include/migraphx/target.hpp 44.44% 5 Missing ⚠️
src/api/api.cpp 81.25% 3 Missing ⚠️
src/include/migraphx/operation.hpp 92.86% 3 Missing ⚠️
src/include/migraphx/context.hpp 50.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4805      +/-   ##
===========================================
+ Coverage    92.49%   92.81%   +0.32%     
===========================================
  Files          583      584       +1     
  Lines        29562    30036     +474     
===========================================
+ Hits         27343    27877     +534     
+ Misses        2219     2159      -60     
Files with missing lines Coverage Δ
src/include/migraphx/allocation_model.hpp 77.78% <ø> (ø)
src/include/migraphx/concat_opt.hpp 88.24% <ø> (ø)
src/include/migraphx/instruction_ref.hpp 100.00% <ø> (ø)
src/include/migraphx/marker.hpp 88.89% <ø> (-0.30%) ⬇️
src/include/migraphx/optional.hpp 100.00% <ø> (ø)
src/include/migraphx/pass.hpp 91.67% <100.00%> (-0.64%) ⬇️
src/include/migraphx/schedule_model.hpp 92.50% <ø> (-0.52%) ⬇️
src/include/migraphx/stream_model.hpp 92.86% <ø> (ø)
src/include/migraphx/context.hpp 70.42% <50.00%> (+1.93%) ⬆️
src/api/api.cpp 73.24% <81.25%> (+1.61%) ⬆️
... and 2 more

... and 31 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
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

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

Comment thread src/api/api.cpp
Comment thread src/include/migraphx/context.hpp
Comment thread src/include/migraphx/stream_model.hpp
Comment thread src/include/migraphx/target.hpp
Comment thread tools/docker/sles.docker
Comment thread src/include/migraphx/marker.hpp
Comment thread src/include/migraphx/schedule_model.hpp
Comment thread src/api/api.cpp
Comment thread src/include/migraphx/concat_opt.hpp
Comment thread src/include/migraphx/pass.hpp
Comment thread CMakeLists.txt Outdated
causten and others added 2 commits April 28, 2026 12:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Chris Austen <causten@users.noreply.github.com>
causten and others added 2 commits April 29, 2026 01:35
Co-Authored-By: Claude Opus 4 (1M context) <noreply@anthropic.com>
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