Set UPAAS_PORT as well as PORT in docker-compose.yml #233

Merged
clawbot merged 1 commits from fix/compose-upaas-port into next 2026-09-29 05:26:52 +02:00
Collaborator

Fixes #230.

upaas reads UPAAS_PORT before PORT. docker-compose.yml only set PORT: "8080", so a UPAAS_PORT line in .env made upaas listen on a port other than 8080, which is the port the compose file's port mapping and healthcheck use, and the container never became healthy.

  • docker-compose.yml: sets UPAAS_PORT: "8080" next to PORT: "8080". The comment above them now says both override .env and that both are needed because upaas reads UPAAS_PORT first.
  • README.md, "Deploying with Docker Compose": the sentence listing the settings the compose file fixes now names PORT, UPAAS_PORT and UPAAS_DATA_DIR. make fmt rewrapped the rest of that paragraph.
  • TODO.md: one Completed Steps entry.

No code changes.

Model: opus-5-5

Fixes https://git.eeqj.de/sneak/upaas/issues/230. upaas reads `UPAAS_PORT` before `PORT`. `docker-compose.yml` only set `PORT: "8080"`, so a `UPAAS_PORT` line in `.env` made upaas listen on a port other than 8080, which is the port the compose file's port mapping and healthcheck use, and the container never became healthy. - `docker-compose.yml`: sets `UPAAS_PORT: "8080"` next to `PORT: "8080"`. The comment above them now says both override `.env` and that both are needed because upaas reads `UPAAS_PORT` first. - `README.md`, "Deploying with Docker Compose": the sentence listing the settings the compose file fixes now names `PORT`, `UPAAS_PORT` and `UPAAS_DATA_DIR`. `make fmt` rewrapped the rest of that paragraph. - `TODO.md`: one Completed Steps entry. No code changes. Model: opus-5-5
clawbot added the needs-review label 2026-09-29 05:09:20 +02:00
clawbot self-assigned this 2026-09-29 05:09:20 +02:00
clawbot added 1 commit 2026-09-29 05:09:20 +02:00
upaas reads UPAAS_PORT before PORT, so a UPAAS_PORT line in .env made it
listen on another port than the one the compose file's port mapping and
healthcheck use, and the container never became healthy. The compose file
now sets both to 8080, and its comment and the README's Docker Compose
section say that both are set.

Model: opus-5-5
Author
Collaborator

Review passed.

Model: opus-5-5

Review passed. Model: opus-5-5
clawbot merged commit 0a94484795 into next 2026-09-29 05:26:52 +02:00
clawbot deleted branch fix/compose-upaas-port 2026-09-29 05:26:52 +02:00
clawbot removed the needs-review label 2026-09-29 05:27:01 +02:00
Sign in to join this conversation.