feat: release release - #12
Merged
Merged
Conversation
Déplace le build Docker des 3 services (api/app/agent) de vigiep1 vers GitHub Actions. Sur vigiep1, bun install (1771+ paquets) prenait 36-41min par service faute de cache incrémental (Dockerfile réinstalle à froid volontairement, cf. incident disque plein du 29/08) et de disque I/O lent sur /mnt/hermes-extra -- un rebuild complet des 3 services dépassait 2h. Le cache GitHub Actions (type=gha) devrait ramener ça à quelques dizaines de secondes sur les runs suivants, sur des runners dédiés (pas de contention RAM/IO avec dagster/n8n/ vigieproc-api qui tournent sur le même host). vigiep1 passera de `docker compose build` à `docker compose pull` -- changement du docker-compose.yml de production (non versionné dans ce repo) à faire séparément, avec provisionnement d'un PAT read:packages pour l'authentification à ghcr.io (registre privé). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LtJccFqpSuP8tQ3Pwg1dYH
|
Retargeted this onto
Nothing is wrong with your branch. If the diff now shows commits that are already on git fetch origin main
git rebase origin/main
git push --force-with-lease |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.github/workflows/build-images.yml: build les 3 images (api,app,agent) sur push versrelease(ou déclenchement manuelworkflow_dispatch), push versghcr.io/franckh-stack/crm-{service}(latest+ tag court du commit).type=gha(cache natif GitHub Actions) — objectif : ramener lebun install(1771+ paquets, 36-41min mesurés sur vigiep1 faute de cache incrémental) à quelques dizaines de secondes sur les runs suivants.apps/appreçoit sesAPI_URL/APP_URLen build-args (https://crm.vigieproc.fr, figés une fois pour toutes — Next.js les grave dans le bundle client au build, pas au runtime, piège déjà rencontré en prod le 29/08/2026).Pourquoi
Sur vigiep1, un rebuild complet des 3 services dépassait 2h (mesuré :
crm-app~45min,crm-agent~70min,crm-api~1h22 au premier build), dominé parbun install --productionsur un disque I/O lent (/mnt/hermes-extra) sans aucun cache réutilisé entre builds (le Dockerfile réinstalle à froid par design, pour éviter l'incident de disque plein du 29/08). Un rebuild en parallèle des 3 services a aussi fait thrasher la RAM/swap de vigiep1 (1,3 Gi libre/7,6 Gi), en contention avec dagster/n8n/vigieproc-api qui tournent sur le même host.Construire sur des runners GitHub dédiés élimine cette contention et permet un vrai cache incrémental.
Ce que cette PR NE fait PAS
docker-compose.ymlde production sur vigiep1 (non versionné dans ce repo) — la basculebuild:→image:+ le provisionnement d'un PATread:packagespour l'authentification àghcr.io(registre privé) restent à faire séparément par Franck.release(ou un déclenchement manuel) lancera le premier run.Test plan
bun install --frozen-lockfile+ push : hook pre-push complet passé (check-types,lint,lint:slop,test, 10/10 tâches turbo, 169/169 testsapps/appdepuis le cache)releaseou viaworkflow_dispatch🤖 Generated with Claude Code
https://claude.ai/code/session_01LtJccFqpSuP8tQ3Pwg1dYH