ci: restore deploy-test and deploy-prod jobs (truncated in #129) #130

Merged

fastie81 merged 2 commits from fix/deploy-jobs-restore into main

2026-09-05 22:35:59 +12:00

fastie81 commented

2026-09-05 22:17:16 +12:00

Owner

Bug

Commit e1fcc24 (merged via #129) accidentally truncated the workflow: a bad index operation in the retention step edit deleted everything from the build job onward, dropping the deploy-test and deploy-prod jobs entirely. As a result the main-merge CI run (303) only ran test/lint/build - no deploy to the test node happened.

Fix

Restores the full 5-job single-image pipeline in ci.yml:

test, lint-frontend, build (single image, ci-)
deploy-test (needs test+lint+build) -> deploys the SAME image to the test node on main and v* tags
deploy-prod (needs test+lint+build+deploy-test) -> promotes the SAME image to prod on v* tags, bound to -p honbu-manager-production

The registry-prune step stays correctly inside the build job (does not truncate the tail).

This is the job definition correction - no behavior change beyond restoring the missing deploy steps.

## Bug Commit e1fcc24 (merged via #129) accidentally truncated the workflow: a bad index operation in the retention step edit deleted everything from the build job onward, dropping the deploy-test and deploy-prod jobs entirely. As a result the main-merge CI run (303) only ran test/lint/build - no deploy to the test node happened. ## Fix Restores the full 5-job single-image pipeline in ci.yml: - test, lint-frontend, build (single image, ci-<sha>) - deploy-test (needs test+lint+build) -> deploys the SAME image to the test node on main and v* tags - deploy-prod (needs test+lint+build+deploy-test) -> promotes the SAME image to prod on v* tags, bound to -p honbu-manager-production The registry-prune step stays correctly inside the build job (does not truncate the tail). This is the job definition correction - no behavior change beyond restoring the missing deploy steps.

fastie81 added 1 commit

2026-09-05 22:17:17 +12:00

ci: restore deploy-test and deploy-prod jobs

CI / test (push) Successful in 2m50s

Details

CI / lint-frontend (push) Successful in 1m14s

Details

CI / build (push) Failing after 2m38s

Details

CI / deploy-test (push) Has been skipped

Details

CI / deploy-prod (push) Has been skipped

Details

3a245d73b9

The retention cleanup change (e1fcc24) accidentally truncated the workflow:
a bad index operation deleted everything from the build job onward, dropping
the deploy-test and deploy-prod jobs entirely - so no deployment happened on
the main merge (run 303 only had test/lint/build).

This restores the full 5-job single-image pipeline (test, lint-frontend,
build, deploy-test, deploy-prod) and keeps the registry-prune step correctly
inside the build job without truncating the tail.

fastie81 added 1 commit

2026-09-05 22:26:59 +12:00

ci: fix retention step - wrong ${{ placeholder broke build

CI / test (push) Successful in 2m50s

Details

CI / lint-frontend (push) Successful in 1m16s

Details

CI / build (push) Successful in 3m45s

Details

CI / deploy-test (push) Has been skipped

Details

CI / deploy-prod (push) Has been skipped

Details

a7fc75e575

The prune step's bash had a literal ${{D}${{ placeholder (leftover from a
chr-$ workaround) which bash tried to expand as unbound variable D under
set -u, failing the build AFTER the image had already pushed. Replaced with
the correct ${{ env.FORGEJO_TOKEN }} expression (runner substitutes it before
bash). Also deduplicated a doubled -H in the curl lines.

fastie81 merged commit bc7d3f5f7b into main

2026-09-05 22:35:59 +12:00

fastie81 referenced this pull request from a commit

2026-09-05 22:36:04 +12:00

Merge pull request 'ci: restore deploy-test and deploy-prod jobs (truncated in #129)' (#130) from fix/deploy-jobs-restore into main