Skip to content

ci: remove AMD GPU container#1434

Merged
sbryngelson merged 1 commit into
MFlowCode:masterfrom
sbryngelson:ci/remove-amd-container
May 13, 2026
Merged

ci: remove AMD GPU container#1434
sbryngelson merged 1 commit into
MFlowCode:masterfrom
sbryngelson:ci/remove-amd-container

Conversation

@sbryngelson
Copy link
Copy Markdown
Member

Summary

  • Removes the AMD GPU (gpu-amd) container from the Docker build workflow and Dockerfile
  • The AMD container required downloading and building AFAR (AMD Flang), installing cmake 3.28, and building a custom MPICH — too much maintenance burden for a container that can't be tested in CI
  • CPU and NVIDIA GPU containers are unaffected

Motivation

The AFAR drop used in the container (rocm-afar-8873-drop-22.2.0) is aging, and newer drops (therock-23.x) have an internally inconsistent hipfort (.mod files compiled with a different compiler), making them unusable without significant per-drop patching. AMD GPU builds on actual clusters (Frontier, hpcfund) continue to work via the system ROCm stack and are not affected by this change.

@qodo-code-review
Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@sbryngelson sbryngelson merged commit e16c79c into MFlowCode:master May 13, 2026
31 checks passed
@sbryngelson sbryngelson deleted the ci/remove-amd-container branch May 13, 2026 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant