Skip to content

Fix server Docker builds for current Rust toolchain and CUDA feature wiring#17

Merged
rwilliamspbg-ops merged 2 commits into
mainfrom
copilot/fix-build-docker-job
Jul 9, 2026
Merged

Fix server Docker builds for current Rust toolchain and CUDA feature wiring#17
rwilliamspbg-ops merged 2 commits into
mainfrom
copilot/fix-build-docker-job

Conversation

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

The build-docker Actions job was failing in the server image build path due to an outdated Rust/Cargo toolchain in the Docker builders. The CUDA image path also referenced a package feature that was never declared at the crate level.

  • Align Docker builders with CI Rust version

    • Update the CPU builder image from Rust 1.75 to Rust 1.87.
    • Install Rust 1.87 explicitly in the CUDA builder instead of relying on Ubuntu-packaged rustc/cargo.
  • Force llama.cpp builds to use Clang in containers

    • Set CC=clang and CXX=clang++ in both server Docker builders so the native llama-cpp build uses the expected compiler toolchain.
  • Declare the crate-level CUDA feature

    • Add a top-level cuda feature in mohawk-server/Cargo.toml and forward it to llama-cpp-2/cuda, matching the existing Dockerfile.cuda build invocation.
  • Net effect

    • The CPU Docker build now uses a toolchain compatible with current crate resolution.
    • The CUDA Docker build no longer fails immediately on an undefined Cargo feature.
[features]
cuda = ["llama-cpp-2/cuda"]

Copilot AI changed the title [WIP] Fix failing GitHub Actions job build-docker Fix server Docker builds for current Rust toolchain and CUDA feature wiring Jul 9, 2026
Copilot AI requested a review from rwilliamspbg-ops July 9, 2026 22:48
@rwilliamspbg-ops rwilliamspbg-ops marked this pull request as ready for review July 9, 2026 23:24
@rwilliamspbg-ops rwilliamspbg-ops merged commit 253d510 into main Jul 9, 2026
8 checks passed
@rwilliamspbg-ops rwilliamspbg-ops deleted the copilot/fix-build-docker-job branch July 10, 2026 01:01
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.

2 participants