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] Pouvoir choisir les fronts qu’on déploit en RA pix #460

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nlepage
Copy link
Member

@nlepage nlepage commented Nov 5, 2024

🍂 Problème

Sur 1024pix/pix on build et on déploit tout le temps tous les fronts, alors qu’on ne les utilise pas forcément tous.

🌰 Proposition

Pouvoir choisir quels fronts on souhaite déployer.

🎃 Remarques

N/A

🪵 Pour tester

Un test concluant a eu lieu sur cette PR : 1024pix/pix#10490

@nlepage nlepage added the cross-team Toutes les équipes de dev label Nov 5, 2024
@nlepage nlepage self-assigned this Nov 5, 2024
@pix-bot-github
Copy link

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

@nlepage nlepage force-pushed the tech-time-choose-front-apps branch 3 times, most recently from a3c4ce1 to 2dd0b37 Compare November 5, 2024 15:43
@nlepage nlepage closed this Nov 19, 2024
@nlepage nlepage reopened this Nov 19, 2024
@pix-bot-github
Copy link

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

@nlepage nlepage closed this Nov 19, 2024
@nlepage nlepage reopened this Nov 19, 2024
@pix-bot-github
Copy link

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

@nlepage nlepage force-pushed the tech-time-choose-front-apps branch 3 times, most recently from e2398ea to b8fb174 Compare November 19, 2024 15:52
@nlepage
Copy link
Member Author

nlepage commented Nov 19, 2024

TODO

  • Ne rien déployer quand rien n’est coché.
  • Mettre en place un debounce de déploiement (qui persiste après un redémarrage de pix-bot).
  • Vérifier que l’auteur du commentaire est bien pix-bot-github
  • 🧹 Penser à supprimer le token d’API "PixBotBuild - Scalingo Review App Manager Test PR 460" sur le compte Scalingo dev.
  • 🧹 Penser à supprimer cette RA dans les settings des webhooks GH.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev Development in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants