#1 fix(deploy): use registry image and correct docker list format

Спојено
jschoubben споји(ла) 1 комит(е) из fix/docker-build-format у main пре 1 месец
jschoubben коментирира пре 1 месец

Problem

The initial scaffold had two bugs that prevented the service from starting:

  1. module.yml docker: format was wrong — used object syntax instead of list syntax, so the builder said “No Docker images to build” and never pushed an image to the registry.

  2. docker-compose.yml used build: . — on ace there's no source code to build from (only the service artifact is deployed). Must reference the registry image.

  3. Wrong certresolver name — used letsencrypt but Traefik is configured with resolver le.

Changes

  • module.yml: docker: → list format (- image: txt-game) matching the snake/f1-circuits pattern
  • docker-compose.yml: build: .image: registry-api.novox.be/novox/txt-game:latest; certresolver=letsencryptcertresolver=le; added container_name; removed unused default network

Test plan

After merge, the pipeline will auto-trigger (webhook is registered). All stages must pass:

  • build → pushes image to registry
  • install/configure/start → ace pulls and runs the container
  • Live check: curl -sSI https://txt.zurag.be returns HTTP 200 with valid LE cert

Task: dd07738f-6e58-4702-86c3-c43b85f236d2

## Problem The initial scaffold had two bugs that prevented the service from starting: 1. **module.yml `docker:` format was wrong** — used object syntax instead of list syntax, so the builder said "No Docker images to build" and never pushed an image to the registry. 2. **docker-compose.yml used `build: .`** — on ace there's no source code to build from (only the service artifact is deployed). Must reference the registry image. 3. **Wrong certresolver name** — used `letsencrypt` but Traefik is configured with resolver `le`. ## Changes - `module.yml`: `docker:` → list format (`- image: txt-game`) matching the snake/f1-circuits pattern - `docker-compose.yml`: `build: .` → `image: registry-api.novox.be/novox/txt-game:latest`; `certresolver=letsencrypt` → `certresolver=le`; added `container_name`; removed unused `default` network ## Test plan After merge, the pipeline will auto-trigger (webhook is registered). All stages must pass: - build → pushes image to registry - install/configure/start → ace pulls and runs the container - Live check: `curl -sSI https://txt.zurag.be` returns HTTP 200 with valid LE cert Task: dd07738f-6e58-4702-86c3-c43b85f236d2
jschoubben затворено пре 1 месец
The pull request has been merged as 026eebae3d.
Пријавите се да се прикључе у овом разговору.
Нема лабеле
Нема фазе
No Assignees
1 учесника
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
Откажи
Сачувај
Још нема садржаја.