Skip to content

fix(docker): repair Build & Push (removed-module COPYs + image name) - #103

Merged
tbruyelle merged 2 commits into
mainfrom
fix/docker-build-push
Jun 22, 2026
Merged

fix(docker): repair Build & Push (removed-module COPYs + image name)#103
tbruyelle merged 2 commits into
mainfrom
fix/docker-build-push

Conversation

@tbruyelle

Copy link
Copy Markdown
Collaborator

Problem

The Build & Push workflow (docker.yml) fails on every main push and on release tags:

ERROR: failed to compute cache key: "/log/go.sum": not found

The Dockerfile still copies go.mod/go.sum for collections, store, and log — non-forked modules that were removed in #22/#25. The build fails on the first missing one. This has been broken since those modules were removed; it surfaced loudly on the v0.500.0 tag.

Fix

  • Remove the three stale COPY <module>/go.mod <module>/go.sum lines. The main module's go mod download doesn't reference them (no local replace to those dirs), and COPY . . still brings the full source for make build.
  • Rename the published image ghcr.io/cosmos/simappghcr.io/atomone-hub/simapp (the cosmos ghcr namespace isn't writable by this org; the build failed before the push, so this was latent). Adjust the name if you'd prefer something else.

CI on this PR runs the docker build (push: false for PRs), so it verifies the Dockerfile builds.

Surfaced while finalizing #90.

…hub/simapp

The Build & Push workflow fails with `"/log/go.sum": not found` because the Dockerfile still copies go.mod/go.sum for collections, store, and log — non-forked modules removed in #22/#25. Remove those COPY lines (the main module's go mod download no longer needs them).

Also rename the published image from ghcr.io/cosmos/simapp to ghcr.io/atomone-hub/simapp, since the cosmos namespace isn't writable by this org (the build failed before reaching the push, so this was latent).
@tbruyelle
tbruyelle merged commit dcd844e into main Jun 22, 2026
@tbruyelle
tbruyelle deleted the fix/docker-build-push branch June 22, 2026 14:36
@tbruyelle tbruyelle mentioned this pull request Jun 22, 2026
4 tasks
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