Skip to content

Build Docker images from source instead of downloading binaries.#2605

Merged
fnando merged 2 commits into
mainfrom
docker-cargo
Jun 4, 2026
Merged

Build Docker images from source instead of downloading binaries.#2605
fnando merged 2 commits into
mainfrom
docker-cargo

Conversation

@fnando
Copy link
Copy Markdown
Member

@fnando fnando commented Jun 4, 2026

What

Build the stellar/stellar-cli Docker image from source via cargo install inside the Dockerfile, and produce a multi-arch image by building each architecture on its own native runner before merging them into a single manifest.

The docker.yml workflow now runs a matrix build job — linux/amd64 on ubuntu-latest and linux/arm64 on ubuntu-24.04-arm — that pushes each platform image by digest, then a merge job assembles those digests into one manifest list (docker buildx imagetools create) under the computed tags. QEMU emulation is no longer used.

Why

The current Docker image is broken (see stellar/stellar-cli-docker#16). The move to building the CLI from source means the image is compiled during the Docker build rather than copied in from a prebuilt binary. Compiling under QEMU emulation for a foreign architecture is prohibitively slow, so each architecture is built on a matching native runner and the results are combined into a single multi-arch manifest at publish time.

This is a temporary solution until stellar/stellar-cli-docker publishes images.

Known limitations

This is a stopgap; image publishing is expected to move to stellar/stellar-cli-docker.

Copilot AI review requested due to automatic review settings June 4, 2026 17:34
@github-project-automation github-project-automation Bot moved this to Backlog (Not Ready) in DevX Jun 4, 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@fnando fnando self-assigned this Jun 4, 2026
@fnando fnando moved this from Backlog (Not Ready) to Needs Review in DevX Jun 4, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d0377a8ebe

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .github/workflows/docker.yml Outdated
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@fnando fnando requested a review from mootz12 June 4, 2026 17:52
@fnando fnando enabled auto-merge (squash) June 4, 2026 18:02
@fnando fnando changed the title Build Docker images from source per architecture and publish a multi-arch manifest Build Docker images from source instead of downloading binaries. Jun 4, 2026
Copy link
Copy Markdown
Contributor

@mootz12 mootz12 left a comment

Choose a reason for hiding this comment

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

LG2M.

Two notes inline

Comment thread Dockerfile
Comment thread .github/workflows/docker.yml Outdated
@fnando fnando merged commit c803cd6 into main Jun 4, 2026
218 checks passed
@fnando fnando deleted the docker-cargo branch June 4, 2026 20:53
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in DevX Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants