Skip to content

fix(ci): poll de previews por estado de deployment (card 52a85645) - #195

Merged
SabaTech-dev merged 5 commits into
mainfrom
fix/preview-poll-deployment-status
Aug 28, 2026
Merged

fix(ci): poll de previews por estado de deployment (card 52a85645)#195
SabaTech-dev merged 5 commits into
mainfrom
fix/preview-poll-deployment-status

Conversation

@SabaTech-dev

Copy link
Copy Markdown
Owner

Segundo fix del pipeline de previews (card 52a85645): el primer deploy real (#182) mostro que el poll abortaba en 'exited:unhealthy' (placeholder pre-start de apps dockerimage recien creadas) antes de que el deployment despachara. Ahora se consulta el estado del DEPLOYMENT via API y solo al finished se exige running + probe. Junto con la rotacion del token a abilities read+write+deploy ('Missing required permissions: deploy' en /start), esto completa el camino a deploy verde.

El app-status 'exited:unhealthy' es el placeholder pre-start en apps dockerimage
recien creadas: el poll anterior abortaba antes de que /start siquiera despachara.
Ahora se consulta GET /deployments/applications/{uuid} (queued/in progress/finished/
failed) y solo al 'finished' se exige app-status running. Run de evidencia: 33192352373.
@github-actions

Copy link
Copy Markdown

PR Check Results

Code Formatting: Passed
Linting: Passed
Type Checking: Failed
Tests: Failed

⚠️ Some checks failed. Please review the details above.

@github-actions

Copy link
Copy Markdown

⚠️ WARN Trivy Security Scan Results

Severity Count
🔴 CRITICAL 0
🟠 HIGH 49
Total 49

Scans performed:

  • 📁 Filesystem (vulnerabilities, secrets, misconfigurations)
  • 🐳 Backend Docker image
  • 🐳 Frontend Docker image
  • 🏗️ IaC misconfiguration detection

⚠️ HIGH vulnerabilities should be addressed. Review required.

📋 View detailed SARIF reports

Download artifacts from this workflow run for full SARIF reports.
Results are also available in the GitHub Security tab.


Trivy v0.71.0 | aquasecurity/trivy

…— faltaba en el merge-ref de esta rama)
@github-actions

Copy link
Copy Markdown

PR Check Results

Code Formatting: Passed
Linting: Passed
Type Checking: Failed
Tests: Failed

⚠️ Some checks failed. Please review the details above.

…:3000 + retry de app-status

- El gate construia la imagen backend con context=. (raiz): COPY requirements.txt
  cogia el de la raiz (framework QA, pytest/bandit) y no el del backend -> sin
  uvicorn/fastapi -> contenedor 'exec: uvicorn not found' (deployment failed, queue 10).
- El probe frontend apuntaba a 5173 pero vite configura 3000 (evidencia: logs del
  contenedor: 'VITE ready, Local: http://localhost:3000/').
- El app-status 'exited:unhealthy' puede persistir por lag de Coolify tras
  'deployment finished': retry 60s; el probe in-container es el check autoritativo.
@github-actions

Copy link
Copy Markdown

PR Check Results

Code Formatting: Passed
Linting: Passed
Type Checking: Failed
Tests: Failed

⚠️ Some checks failed. Please review the details above.

@github-actions

Copy link
Copy Markdown

PR Check Results

Code Formatting: Passed
Linting: Passed
Type Checking: Failed
Tests: Failed

⚠️ Some checks failed. Please review the details above.

…-fatales

El comment step moria con 403 'Resource not accessible by integration' (GITHUB_TOKEN
default read-only en este repo) DESPUES de un deploy correcto, teñiendo de rojo un
preview sano. Ahora: permissions minimos por job y el paso cosmico de comentario
es continue-on-error (la verdad del deploy es la API + probe, no el comentario).
@github-actions

Copy link
Copy Markdown

PR Check Results

Code Formatting: Passed
Linting: Passed
Type Checking: Failed
Tests: Failed

⚠️ Some checks failed. Please review the details above.

@github-actions

Copy link
Copy Markdown

🔧 Backend Preview Deployment

App Coolify: qa-framework-backend-pr-195 (wcu0gbbotksswgm7ie1rm2rd)
Estado: ⚠️ running:unknown
Commit: 5677259d51dd4f6568cdaa47b131b63371c830bb
Branch: fix/preview-poll-deployment-status

Preview interna (sin FQDN público hasta decisión de proxy — card 52a85645).

@github-actions

Copy link
Copy Markdown

🎨 Frontend Preview Deployment

App Coolify: qa-framework-frontend-pr-195 (a7un3gghhvipfl9wnylibidy)
Estado: ⚠️ running:unknown
Commit: 5677259d51dd4f6568cdaa47b131b63371c830bb
Branch: fix/preview-poll-deployment-status

Preview interna (sin FQDN público hasta decisión de proxy — card 52a85645).

@SabaTech-dev
SabaTech-dev merged commit 8a56023 into main Aug 28, 2026
39 checks passed
@github-actions

Copy link
Copy Markdown

🧹 Preview Cleanup

PR #195 - Merged

✅ Preview apps eliminadas de Coolify (verificado via API): backend + frontend.


Resources freed at 2026-08-28T18:29:48.842Z

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant