release: deploy-web-skip rapporteert als groen 'success' (misleidend) #1169

New issue

Closed

opened 2026-08-03 15:12:25 +00:00 by brenno · 0 comments

brenno commented

2026-08-03 15:12:25 +00:00

Owner

In .forgejo/workflows/release.yml slaat de job 'Webversie live zetten' (deploy-web) de live-stap over wanneer DEPLOY_SSH_KEY/DEPLOY_KNOWN_HOSTS ontbreken — dat is de bedoeling (deployen gebeurt met de hand). Maar de job-conclusie blijft gewoon success/groen; de uitleg staat alleen in de staplog. Wie de run bekijkt, ziet 'Webversie live zetten ✓' en concludeert redelijkerwijs dat het web live is. Bij de v0.2.0-cut bevestigde version.json op ocideck.librekat.nl nog 0.1.3 terwijl de job groen stond — ik rapporteerde bijna dat het web live was.

Voorstel: de skip zichtbaar onderscheiden — bv. een regel naar $GITHUB_STEP_SUMMARY ('⏭️ NIET live gezet — handmatig via make deploy-web') zodat het op de run-samenvatting staat, en/of een duidelijke stapnaam. Overweeg een post-deploy live-versie-assertie wanneer er wél gedeployd wordt.

Wordt (deels) opgepakt in de bijbehorende verbeter-PR.

In `.forgejo/workflows/release.yml` slaat de job 'Webversie live zetten' (deploy-web) de live-stap over wanneer `DEPLOY_SSH_KEY`/`DEPLOY_KNOWN_HOSTS` ontbreken — dat is de bedoeling (deployen gebeurt met de hand). Maar de job-conclusie blijft gewoon **success/groen**; de uitleg staat alleen in de staplog. Wie de run bekijkt, ziet 'Webversie live zetten ✓' en concludeert redelijkerwijs dat het web live is. Bij de v0.2.0-cut bevestigde `version.json` op ocideck.librekat.nl nog 0.1.3 terwijl de job groen stond — ik rapporteerde bijna dat het web live was. Voorstel: de skip zichtbaar onderscheiden — bv. een regel naar `$GITHUB_STEP_SUMMARY` ('⏭️ NIET live gezet — handmatig via make deploy-web') zodat het op de run-samenvatting staat, en/of een duidelijke stapnaam. Overweeg een post-deploy live-versie-assertie wanneer er wél gedeployd wordt. Wordt (deels) opgepakt in de bijbehorende verbeter-PR.

brenno added the

enhancement

label

2026-08-03 15:12:25 +00:00

brenno referenced this issue from a pull request that will close it,

2026-08-03 15:18:10 +00:00

ci(release): gate-caches delen, deploy-web robuust, web-skip zichtbaar #1172

brenno referenced this issue from a commit

2026-08-03 15:18:11 +00:00

ci(release): deel gate-caches, maak deploy-web robuust, toon web-skip

brenno closed this issue

2026-08-03 15:24:24 +00:00