📝(release) update release.md #829
Conversation
|
|
||
| 7. The release is now done! |
There was a problem hiding this comment.
En fait le 3, 4, 5, 6, 7 (ligne 32 à 46) devraient etre la suite du "4. Commit your changes with the following format:". Il a eu un foirage :D
Il s'agit de la description de ce que fait le script de release.
Il y a besoin de décrire en détails ce qu'il fait. Le déploiement en preprod se fait via le script "make release" pas à la main. Donc faudrait laisser les explications ici.
Et le "3. Open a pull request ask you to wait for an approval from your peers and merge it." devrait être un truc du genre "Create a new and ask you to create a PR..."
There was a problem hiding this comment.
Ah yes merci pour le rappel, j'avais oublié que le script ouvrait automatiquement une branche là-bas.
Ca donne un peu envie d'ouvrir aussi la PR de prod à ce moment-là aussi, non ? Elle peut rester une semaine.
Peut-être même qu'on pourrait la mettre en automerge et ce serait JB qui, une fois sa recette faite, met le approve et déclencherait la prod. Qu'est-ce que tu en dis ?
There was a problem hiding this comment.
Surtout pas de branche auto généré pour la prod. La MEP doit se faire manuellement par un dév. Ce n'est que 2 ligne d'un fichier à changer. Ya trop de risque à mettre de l'automagique.
mjeammet
force-pushed
the
mpj/update-release-md
branch
5 times, most recently
from
7efccb5 to
5ff5d28
Compare
Update documentation on release
mjeammet
force-pushed
the
mpj/update-release-md
branch
from
5ff5d28 to
118932c
Compare
| > [!NOTE] | ||
| > It also open the PR for pre-prod deployment, see following section. | ||
|
|
||
| 3. Following release script instructions, tag and push your commit: |
There was a problem hiding this comment.
C'est dans le script aussi les instructions pour le tag du coup ça me fait bizarre cette phrase.
En fait le script de release il est super détaillé et fait tout. Il suffit de suivre les instructions du script. Est-ce que ça ne vaudrait pas le coup ici de mettre des captures d'écran du terminal pour voir ce qui se passe ? (c'est peut être un peu trop :D)
Ya juste un truc à savoir c'est de bien avoir le dépôt de lasuite-deploiement cloné pour que le script fasse aussi la PR de déploiement en preprod.
Est-ce que du coup ça serait intéressant de mettre un warning pour dire attention il faut avoir le dépôt cloné avec les droits pour que le script fasse le taff jusqu'au bout ?
There was a problem hiding this comment.
En fait, même si le script le dit, je pense qu'on doit le dire ici parce que ça fait partie des actions à entreprendre.
OK je rajoute un mot sur le clonage du repo "deploiement". Pour info, j'essaie de faire en sorte que ce soit clair qu'on a deux étapes (d'abord on crée l'image, ensuite on la déploie sur argocd). Comme pour nous ça n'a pas été tout de sute très clair, j'ai envie que ça le soit pour de bon dans ce fichier.
Purpose
Update documentation on release process.