Skip to content

[IN-200]: ci: run tests inside production image - #233

Merged
apsantiso merged 4 commits into
masterfrom
ci/tests-in-production-image
Nov 6, 2026
Merged

apsantiso merged 4 commits into
masterfrom
ci/tests-in-production-image

Conversation

@apsantiso

@apsantiso apsantiso commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Why

Production deployments are not working. The last successful build & deploy on master was on 2026-08-19.

The last four merges failed and did not reach production:

Date Commit PR
2026-10-19 d7c9fa51 #230
2026-11-05 21892ba4 #217
2026-11-06 5cd06764 #231
2026-11-06 c37bd41e #232

CI did not catch it because tests run on the runner and never build the Dockerfile.

Changes

  • Dockerfile, infrastructure/preview.Dockerfile, infrastructure/development.Dockerfile: use node:16 and drop the apt step. The image already includes the build tools.
  • ci.yml: build the production image on PRs and run unit and e2e tests inside it. No longer runs on pushes to master.
  • docker-compose.test.yml, .env.test: test stack and fake env values, moved out of ci.yml.

Notes

  • Merging this deploys everything pending from the four PRs above at once.

bullseye packages return 404. node:16 already ships build tools.
Move test stack and env to compose. Exclude .env files from image.
@apsantiso
apsantiso marked this pull request as ready for review November 6, 2026 14:25
@apsantiso
apsantiso requested a review from sg-gs as a code owner November 6, 2026 14:25
@apsantiso apsantiso assigned apsantiso and unassigned apsantiso Nov 6, 2026
@apsantiso apsantiso changed the title ci: run tests inside production image [IN-200]: ci: run tests inside production image Nov 6, 2026
@apsantiso apsantiso added the bug Something isn't working label Nov 6, 2026
Comment thread docker-compose.test.yml Resolved
@apsantiso
apsantiso requested a review from sg-gs November 6, 2026 14:50
@apsantiso
apsantiso merged commit fdd820c into master Nov 6, 2026
6 of 7 checks passed
@apsantiso
apsantiso deleted the ci/tests-in-production-image branch November 6, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ready-for-preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants