Merge pull request 'ci: construir antes de bajar el servicio' (#2) from fix/deploy-robusto into main

2026-08-03 20:04:35 -03:00
parent 8d1b5da448 f1ef9da857
commit d74c97e532
1 changed files with 26 additions and 2 deletions
@@ -10,9 +10,33 @@ jobs:
    steps:
      - name: Pull y rebuild
        run: |
+          set -eu
          git config --global --add safe.directory /home/proyecto/taller_automotriz
          cd /home/proyecto/taller_automotriz
          git fetch origin
          git reset --hard origin/main
-          docker compose down
-          docker compose up -d --build
+
+          # Construir ANTES de tocar lo que esta corriendo. Antes este paso
+          # hacia "docker compose down" y recien despues "up --build": si la
+          # construccion fallaba (por ejemplo, npm sin poder bajar las
+          # dependencias), el taller quedaba caido hasta que alguien lo
+          # notara. Pasó el 03/08/2026.
+          docker compose build
+
+          # Con la imagen nueva ya lista, "up -d" recrea los contenedores que
+          # cambiaron. No hace falta bajar nada de antemano.
+          docker compose up -d --no-build --remove-orphans
+
+      - name: Verificar que quedo arriba
+        run: |
+          set -eu
+          cd /home/proyecto/taller_automotriz
+          docker compose ps
+
+          # Si el contenedor no esta corriendo, el despliegue fallo aunque los
+          # comandos anteriores hayan salido bien.
+          if [ -z "$(docker compose ps --quiet --status running)" ]; then
+            echo "El despliegue termino sin contenedores corriendo"
+            docker compose logs --tail 50
+            exit 1
+          fi