Skip to content

fix(ci): wire AMDGPU_TARGETS through backend build workflow - #9445

Merged
mudler merged 1 commit into
mudler:masterfrom
russell:fix-amdgpu-targets
Apr 20, 2026
Merged

fix(ci): wire AMDGPU_TARGETS through backend build workflow#9445
mudler merged 1 commit into
mudler:masterfrom
russell:fix-amdgpu-targets

Conversation

@russell

@russell russell commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Commit 8839a71 exposed AMDGPU_TARGETS as an ARG/ENV in Dockerfile.llama-cpp so GPU targets could be overridden, but never wired the value through the CI workflow inputs. Without it, Docker receives AMDGPU_TARGETS="" which overrides the Makefile's ?= default, causing all hipblas builds to compile only for gfx906 regardless of the target list in the Makefile.

Add amdgpu-targets as a workflow_call input with the same default list as the Makefile, and pass it as AMDGPU_TARGETS in the build-args of both the push and PR build steps.

Assisted-by: Claude Code:claude-sonnet-4-6

Signed commits

  • Yes, I signed my commits.

Commit 8839a71 exposed AMDGPU_TARGETS as an ARG/ENV in
Dockerfile.llama-cpp so GPU targets could be overridden, but never
wired the value through the CI workflow inputs. Without it, Docker
receives AMDGPU_TARGETS="" which overrides the Makefile's ?= default,
causing all hipblas builds to compile only for gfx906 regardless of
the target list in the Makefile.

Add amdgpu-targets as a workflow_call input with the same default list
as the Makefile, and pass it as AMDGPU_TARGETS in the build-args of
both the push and PR build steps.

Assisted-by: Claude Code:claude-sonnet-4-6
Signed-off-by: Russell Sim <rsl@simopolis.xyz>
@russell
russell force-pushed the fix-amdgpu-targets branch from f69d020 to 399c1de Compare April 20, 2026 17:01

@mudler mudler left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

good catch - thank you!

@mudler
mudler merged commit c66c41e into mudler:master Apr 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants