ci: construir antes de bajar el servicio #2

2026-08-03T19:51:12-03:00

jpbeldomenico commented

2026-08-03 19:51:12 -03:00

Que paso hoy

El despliegue hacia docker compose down y recien despues up -d --build. Hoy 03/08 la construccion fallo porque npm no pudo bajar las dependencias:

npm error network

Resultado: el taller quedo caido — sin contenedores — y ningun paso del pipeline lo aviso. El job figuraba como fallido, pero el servicio estuvo abajo hasta que se levanto a mano con la imagen anterior.

El arreglo

Construir primero, sin tocar lo que corre. Si la construccion falla, el servicio sigue en pie y el despliegue simplemente no ocurre.
Sin down previo: con la imagen nueva lista, up -d recrea solo lo que cambio.
Verificacion final: si al terminar no hay contenedores corriendo, el paso falla y muestra los registros, en vez de dar por bueno un servicio caido.

Es el mismo criterio que ya usa el despliegue de prode_app, que construye, levanta y recien entonces espera el healthcheck.

## Que paso hoy El despliegue hacia `docker compose down` y **recien despues** `up -d --build`. Hoy 03/08 la construccion fallo porque npm no pudo bajar las dependencias: ``` npm error network ``` Resultado: **el taller quedo caido** — sin contenedores — y ningun paso del pipeline lo aviso. El job figuraba como fallido, pero el servicio estuvo abajo hasta que se levanto a mano con la imagen anterior. ## El arreglo 1. **Construir primero**, sin tocar lo que corre. Si la construccion falla, el servicio sigue en pie y el despliegue simplemente no ocurre. 2. **Sin `down` previo**: con la imagen nueva lista, `up -d` recrea solo lo que cambio. 3. **Verificacion final**: si al terminar no hay contenedores corriendo, el paso falla y muestra los registros, en vez de dar por bueno un servicio caido. Es el mismo criterio que ya usa el despliegue de prode_app, que construye, levanta y recien entonces espera el healthcheck.

jpbeldomenico added 1 commit 2026-08-03 19:51:14 -03:00

fix(ci): construir antes de bajar el servicio

Revisión con IA / ai-review (pull_request) Failing after 24s

Details

f1ef9da857

El despliegue hacia "docker compose down" y recien despues
"up -d --build". Si la construccion fallaba, el taller quedaba caido hasta
que alguien lo notara: paso el 03/08/2026, cuando npm no pudo bajar las
dependencias y el sitio estuvo abajo sin que ningun paso lo avisara.

Ahora la imagen se construye primero y solo despues se recrean los
contenedores, sin bajar nada de antemano. Si la construccion falla, lo que
estaba corriendo sigue en pie.

Se agrega ademas una verificacion final: si al terminar no hay contenedores
corriendo, el despliegue falla y muestra los registros, en lugar de dar por
bueno un servicio caido.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

jpbeldomenico merged commit d74c97e532 into main

2026-08-03 20:04:37 -03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jpbeldomenico/taller_automotriz#2