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] Ajoute la commande de création, publication et déploiement en recette d'un patch (hotfix). #73

Merged
merged 3 commits into from
May 12, 2021

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented May 6, 2021

🦄 Problème

La création d‘un hotfix se fait actuellement entièrement manuellement.
Par définition, le hotfix est nécessaire dans des conditions de stress inhabituelles.
Dépendre d'une opération manuelle est une source de stress additionnelle.

🤖 Solution

Pour la santé mentale de l‘équipe technique, on ajoute une commande slack /deploy-hotfix [branch-name].
Cette commande appelle un endpoint de Pix-bot qui se charge de créée une nouvelle release de type patch à partir de la branche donnée.
Après publication, la release est déployée en recette.

🌈 Remarques

Après la recette du hotfix, on peut le déployer en production en utilisant Slack comme à l'accoutumée.
Il suffira de préciser le nom du hotfix comme nom de version à mettre en production.

Il faudra par la suite créer une Pull-Request depuis la branche de hotfix vers dev pour que la branche principale soit à jour avec le hotfix.

💯 Pour tester

En local uniquement.
Lancer Pix-bot avec npm start.
Lancer ngrock avec ngrock http 3000.
Configurer la commande /deploy-hotfix de l'environnement Slack Pix-Bot-Test avec l'adresse donnée par ngrock suivi de /slack/commands/create-and-deploy-pix-hotfix.
Taper la commande /deploy-hotfix suivie du nom de la branche à déployer.
Vérifier la création d'un commit [RELEASE] A patch is being released to x.y.z. avec z > 0.

@HEYGUL HEYGUL requested a review from bpetetot May 6, 2021 11:07
@HEYGUL HEYGUL added cross-team Toutes les équipes de dev Tech Review Needed labels May 6, 2021
Copy link
Contributor

@MathieuGilet MathieuGilet left a comment

Choose a reason for hiding this comment

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

Le code me semble bon mais je n'ai pas testé fonctionnellement car je n'ai pas de connaissances sur Pix Bot Test.

@HEYGUL HEYGUL marked this pull request as draft May 6, 2021 15:21
@HEYGUL HEYGUL changed the title Add branch name as publish.sh argument [FEATURE] Ajoute la commande de création, publication et déploiement en recette d'un patch (hotfix). May 6, 2021
@HEYGUL HEYGUL marked this pull request as ready for review May 6, 2021 16:10
common/controllers/slack.js Show resolved Hide resolved
common/services/releases.js Outdated Show resolved Hide resolved
@bpetetot bpetetot force-pushed the feature-add-hotfix-publication branch from ef620de to 828ab1b Compare May 12, 2021 13:12
@bpetetot bpetetot merged commit 232f3ef into main May 12, 2021
@bpetetot bpetetot deleted the feature-add-hotfix-publication branch May 12, 2021 13:15
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants