Skip to content

refactor(container): name the compose service by role - #45

Open
jravani wants to merge 1 commit into
enerplanetfrom
refactor/compose-naming
Open

jravani wants to merge 1 commit into
enerplanetfrom
refactor/compose-naming

Conversation

@jravani

@jravani jravani commented Oct 30, 2026

Copy link
Copy Markdown
Member

Summary

  • infrastructure/container/docker-compose.yml pins the project to buem and renames the service buem-api to api with no container_name; the container is buem-api-1.
  • The service names the published image (ghcr.io/enerplanet/buem-model:${BUEM_MODEL_IMAGE_TAG}), so the same file runs a release with --pull always --no-build or builds the checkout with --build.
  • Setup scripts and docs use docker compose ... exec api / logs api. The publishing workflow comment points at buem-gateway's merged environment/docker-compose.yml.

Known issue, not changed here

With this compose file the 6.10.0 image does not become healthy: src/buem/data/weather is mounted read-only, and the server creates location_cache inside it on start (OSError: [Errno 30] Read-only file system). The mount predates this change; buem-gateway's compose file does not mount it and runs the same image healthy.

Testing

  • config -q passes; config --services gives api.
  • Run with 6.10.0: container buemmtest-api-1 created as expected, then unhealthy for the reason above.

infrastructure/container/docker-compose.yml pins the project to `buem`
and renames the service `buem-api` to `api` with no container_name, so
the container is buem-api-1. The service also names the published
image (ghcr.io/enerplanet/buem-model, BUEM_MODEL_IMAGE_TAG), so the
same file runs a release with `--pull always --no-build` or builds this
checkout with `--build`.

The setup scripts and docs use `docker compose ... exec api` and
`logs api`. The publishing workflow comment points at buem-gateway's
merged environment/docker-compose.yml.
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