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

병합
jschoubben fix/docker-build-format 에서 main 로 1 commits 를 머지했습니다 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
The pull request has been merged as 026eebae3d.
로그인하여 이 대화에 참여
레이블 없음
마일스톤 없음
담당자 없음
참여자 1명
마감일

마감일이 설정되지 않았습니다.

의존성

이 풀 리퀘스트는 어떠한 의존성도 가지지 않습니다.

불러오는 중...
취소
저장
아직 콘텐츠가 없습니다.