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

Recettes jetables: utilisation de git #4769

Merged
merged 3 commits into from
Sep 19, 2024

Conversation

xavfernandez
Copy link
Contributor

🤔 Pourquoi ?

La création de recettes jetables est très lente.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@xavfernandez xavfernandez added the no-changelog Ne doit pas figurer dans le journal des changements. label Sep 18, 2024
@xavfernandez xavfernandez self-assigned this Sep 18, 2024
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 18, 2024
@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 18, 2024
@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 18, 2024
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 18, 2024
@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 18, 2024
@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 18, 2024
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 18, 2024
@xavfernandez xavfernandez marked this pull request as ready for review September 18, 2024 10:15
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

Copy link
Contributor

@rsebille rsebille left a comment

Choose a reason for hiding this comment

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

10 minutes 👏.

Je verrais bien la partie qui remplace la commande deploy dans un script shell externe, ça éviterais la répétition, ça permettrais de l'utiliser dans create-fast-machine.sh, et ça serais linté par shellcheck.

$CLEVER_CLI deploy --branch $BRANCH --force
git remote add review_app $($CLEVER_CLI applications --json | jq --raw-output .[0].deploy_url)
git remote -v
GIT_ASKPASS=$(mktemp) && chmod u+rx $GIT_ASKPASS && export GIT_ASKPASS
Copy link
Contributor

Choose a reason for hiding this comment

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

GIT_ASKPASS 🤯

.github/workflows/review-app-creation.yml Outdated Show resolved Hide resolved
@xavfernandez xavfernandez force-pushed the xfernandez/faster_review_apps branch 2 times, most recently from b4845bf to 7e07ab7 Compare September 18, 2024 12:58
Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

J’ai toujours un peu espoir qu’une implémentation Python au lieu de bash soit plus lisible et maintenable, mais bon, on peut se dépatouiller du bash.

Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

❤️

scripts/clever-deploy Outdated Show resolved Hide resolved
scripts/clever-deploy Outdated Show resolved Hide resolved
@xavfernandez xavfernandez removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 18, 2024
@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 18, 2024
@xavfernandez
Copy link
Contributor Author

J'ai rajouté l'option --progress qui est certes un peu verbeuse mais qui permet de savoir à peu près où on en est...

Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@xavfernandez
Copy link
Contributor Author

Et comme discuté avec François, quand le worker GA est en forme, il est capable de git push en 2 minutes: https://github.com/gip-inclusion/les-emplois/actions/runs/10924818541/job/30324809682, parfois avant que Postgresql ne soit dispo et 💥

=> Il faudrait donc remettre sur la table la management command wait_for_db de #4523

Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Top !

@xavfernandez xavfernandez added this pull request to the merge queue Sep 19, 2024
Merged via the queue into master with commit 0587e37 Sep 19, 2024
20 of 21 checks passed
@xavfernandez xavfernandez deleted the xfernandez/faster_review_apps branch September 19, 2024 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants