Skip to content

fix(docker): make docker-compose runnable standalone#49

Merged
offendingcommit merged 1 commit into
mainfrom
fix/docker-compose-standalone
May 29, 2026
Merged

fix(docker): make docker-compose runnable standalone#49
offendingcommit merged 1 commit into
mainfrom
fix/docker-compose-standalone

Conversation

@offendingcommit
Copy link
Copy Markdown
Owner

Problem

docker compose up failed out of the box:

service "openconcho" depends on undefined service "api": invalid compose project

The compose was written as a snippet to drop into a Honcho stack (which provides api), but as a standalone file the depends_on: api is invalid.

Fix

  • Drop depends_on: api — the blocker.
  • Default HONCHO_UPSTREAM to http://host.docker.internal:8000 (overridable: HONCHO_UPSTREAM=… docker compose up) so it points at a Honcho on the host with zero config.
  • extra_hosts: host.docker.internal:host-gateway so that default resolves on Linux too (Docker Desktop/Colima already provide it).
  • Combined-stack instructions moved to comments.

Verified locally

docker compose config validates; docker compose up starts the container, /healthz returns ok, config.js injects same-origin. No undefined-service error.

Patch (fix) → cuts v0.13.1, which re-publishes the image to the now-linked GHCR package.

The service declared depends_on the Honcho 'api' service, which doesn't
exist in this standalone compose — 'docker compose up' failed with
'depends on undefined service api'. Drop the dependency, default
HONCHO_UPSTREAM to the host's Honcho (host.docker.internal:8000,
overridable), and add an extra_hosts mapping so the default also
resolves on Linux. Combined-stack instructions moved to comments.
@offendingcommit offendingcommit merged commit 9ab9d52 into main May 29, 2026
2 checks passed
@offendingcommit offendingcommit deleted the fix/docker-compose-standalone branch May 29, 2026 16:49
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.

1 participant