Skip to content

Update main.yml#32

Merged
Tony-Rome merged 2 commits intomainfrom
pch-change-31-archivo-yml
May 1, 2025
Merged

Update main.yml#32
Tony-Rome merged 2 commits intomainfrom
pch-change-31-archivo-yml

Conversation

@Tony-Rome
Copy link
Copy Markdown
Member

@Tony-Rome Tony-Rome commented May 1, 2025

Propuesta actualización archivo yaml para aprovechar recursos de github actions y quitar responsabilidad en droplet.

En este documento me basé para la actualización.

Falta verificar:

  • Ruta /home/pyday que exista en el lado de droplet.
  • pm2 instalado, sino ejecutar npm install pm2.
  • Node versión 20 instalado, sino ejecutar curl -fsSL https://deb.nodesource.com/setup_20.x | sudo -E bash -
    sudo apt install -y nodejs
    .

En lo personal sé pocas cosas de node por lo que dejo algunas observaciones, por si alguien tiene también dudas:

  • npm ci es parecido a npm install pero para ambientes automatizados, por lo que calza para este caso, dejo la docu oficial del comando.
  • target: "/home/pyday" es del lado de droplet.
  • pm2 necesario para SSR acá la Docu oficial.
  • npm install --omit=dev es necesario para node_modules y para omitir instalar cosas de desarrollo.
  • pm2 delete pyday || true borrado del app forzada para evitar cualquier problema ante una excepción lanzada.
  • pm2 start npm --name "pyday" -- start me basé en la docu de marifer y lili.
  • Importante Como única vez después del primer deploy hay que ejecutar pm2 startup y pm2 save para el auto-inicio ante cualquier reinicio.

Cualquier cosa errónea lo actualizaré e igual pueden subir sus cambios acá si es que yo me demoro. Gracias!

@Tony-Rome Tony-Rome requested review from a team, BGMP, Ferx096, Jamadath, KrlitosForever, LiliGC, MariferVL, ancaneo, pablolirag and yeriel and removed request for a team May 1, 2025 22:20
Copy link
Copy Markdown
Member

@KrlitosForever KrlitosForever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confio!

@Tony-Rome Tony-Rome merged commit c9a89f0 into main May 1, 2025
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.

2 participants