Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Ajouter un check Github lors du déploiement des review apps (PIX-14768). #461

Merged
merged 10 commits into from
Nov 8, 2024

Conversation

MathieuGilet
Copy link
Contributor

@MathieuGilet MathieuGilet commented Nov 6, 2024

🍂 Problème

Aujourd'hui, un non déploiement d'une ou plusieurs review apps ne bloque pas le merge de la PR. L'ajout d'un check required n'est pas possible en l'état car les status check rajoutés par Scalingo lors du déploiement d'une ou plusieurs review app n'a pas un nom unique.

🌰 Proposition

  • Ajouter un check Github au statut pending lors du déploiement d'une RA.
  • Enregistrer dans une base pg le déploiement de cette RA (nous avons de conserver un état pour la suite).
  • Ajouter un endpoint, qui sera appelé par Scalingo lors du déploiement d'une RA. Cet endpoint va:
    • Si le déploiement a réussi:
      • Mettre à jour la RA en base pour dire qu'elle est déployée.
      • Passer le check au statut success lorsque toutes les RA d'une même PR sont déployées.
    • Si le déploiement a échoué:
      • Mettre à jour la RA en base pour dire qu'elle n'est pas déployée.
      • Passer le check au statut failure.

🎃 Remarques

Un fois cette PR mergée il faudra:

  • Rajouter un webhook sur l'event deployment dans Scalingo.
  • Rendre le check required dans la configuration du projet dans Github.

@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr461.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-bot-review-pr461/environment

@MathieuGilet MathieuGilet force-pushed the pix-14768-add-check-on-ra-deployment branch 2 times, most recently from 04e73b3 to f71101a Compare November 6, 2024 16:06
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr461.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-bot-review-pr461/environment

@MathieuGilet MathieuGilet force-pushed the pix-14768-add-check-on-ra-deployment branch 4 times, most recently from 6452401 to 84d5fc7 Compare November 7, 2024 15:02
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr461.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-bot-review-pr461/environment

@MathieuGilet MathieuGilet force-pushed the pix-14768-add-check-on-ra-deployment branch from 84d5fc7 to 460772b Compare November 7, 2024 15:10
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr461.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-bot-review-pr461/environment

1 similar comment
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr461.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-bot-review-pr461/environment

@github-actions github-actions bot force-pushed the pix-14768-add-check-on-ra-deployment branch from b88d045 to 3e7b485 Compare November 8, 2024 15:03
@github-actions github-actions bot merged commit ceab6a3 into main Nov 8, 2024
2 of 3 checks passed
@github-actions github-actions bot deleted the pix-14768-add-check-on-ra-deployment branch November 8, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Ready to Merge team-captains This is your captain speaking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants