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] Permettre le déploiement dès la création d'une release (PIX-13870). #430

Merged
merged 3 commits into from
Aug 20, 2024

Conversation

VincentHardouin
Copy link
Member

@VincentHardouin VincentHardouin commented Aug 13, 2024

🦄 Problème

Actuellement, plusieurs étapes manuelles persistent sur certains repository comme la création de release via l'interface de GitHub et le déploiement depuis une commande Slack. Les deux sont toujours suivies, cela n'a donc pas d'intérêt d'être manuel.

🤖 Proposition

Dans cette PR, nous nous concentrons sur la deuxième étape qui est de remplacer la commande Slack par l'utilisation des Webhooks GitHub.

GitHub appelle déjà Pix Bot à chaque création de release mais Pix Bot ne fait rien de cet évènement. Ici, nous ajoutons donc la logique pour que Pix Bot puisse déployer les apps associées au repo qui a une nouvelle release.

🌈 Remarques

Cette PR vient avec l'ajout d'une nouvelle variable d'environnement. REPO_APP_NAMES_MAPPING qui a pour but d'éviter les PRs pour ajouter une fonctionnalité pour un nouveau repository comme par exemple : #212.

Actuellement, l'écoute des webhooks se fait dans le dossier/app build, mais dans le cadre des releases ça devrait être dans le run. Je propose de voir ça dans un second temps, pour ne pas complexifier le code de cette PR en dupliquant la route etc.

💯 Pour tester

Aucune idée pour brancher ça facilement

@pix-bot-github
Copy link

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

@Gudsfile Gudsfile marked this pull request as ready for review August 13, 2024 15:20
@VincentHardouin VincentHardouin changed the title [FEATURE] Permettre le déploiement dès la création d'une release. [FEATURE] Permettre le déploiement dès la création d'une release (PIX-13870). Aug 14, 2024
@Gudsfile Gudsfile force-pushed the add-deploy-on-release branch from b6430ee to 4b96715 Compare August 14, 2024 09:31
@yannbertrand
Copy link
Member

Ça concerne quel(s) projet(s) ? J'ai pas le souvenir d'avoir créé de release GitHub manuellement de mon côté

@yannbertrand
Copy link
Member

Pas de conflit à prévoir sur les projets qui utilisent la release semantic ?

@VincentHardouin
Copy link
Member Author

Ça concerne quel(s) projet(s) ? J'ai pas le souvenir d'avoir créé de release GitHub manuellement de mon côté

Presque tous les projets data sont en release manuel, mais c'est plutôt l'après release, le déploiement, qu'on souhaite résoudre ici. C-a-d après que la release soit faite que celle-ci soit déployée automatiquement, donc pas de conflit avec semantic-release

@github-actions github-actions bot merged commit 342d958 into main Aug 20, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants