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

Portage Docker/Python vers js pour la GH Action ⏱ #13

Closed
Dougniel opened this issue Aug 22, 2022 · 9 comments · Fixed by #26
Closed

Portage Docker/Python vers js pour la GH Action ⏱ #13

Dougniel opened this issue Aug 22, 2022 · 9 comments · Fixed by #26
Assignees
Labels
ci Continous Integration devops enhancement New feature or request released RUN

Comments

@Dougniel
Copy link
Member

Dougniel commented Aug 22, 2022

on a aussi le ticket d'entrée Docker sur GH Action

Peux-tu le créer stp et le linker à celui-ci ?

Originally posted by @adriens in #11 (comment)

  • 15s est le temps que met cette GitHub action pour le Setup à chaque exécution et ça semble être liée au fait que ce soit une image Docker
  • Ce temps peut faire basculer une exécution à la minute de facturation GitHub supérieur 💰
@Dougniel Dougniel added enhancement New feature or request ci Continous Integration RUN labels Aug 22, 2022
@Dougniel
Copy link
Member Author

@adriens, j'ai mon todo un peu vide (et ça risque de rester un peu vu que tu n'es pas là la semaine pro) et pas mal en pending, je peux passer sur ce ticket ? Sauf si tu vois d'autres priorités

@adriens
Copy link
Member

adriens commented Oct 27, 2022

Yes, of course, profite-en. Il faudra par-contre s'occupper de https://github.com/opt-nc/GLIA/issues/849 🙏 😓 ... mais en mocde super clean avec picocli

@adriens adriens added the devops label Oct 27, 2022
@adriens adriens pinned this issue Oct 27, 2022
@adriens
Copy link
Member

adriens commented Oct 27, 2022

Issue pinée

image

@adriens adriens assigned Dougniel and unassigned adriens Oct 27, 2022
Dougniel added a commit that referenced this issue Nov 7, 2022
- même pricipe que la version Python : _une requête de lancement et on scrute ensuiite l'état du job tower_
- test unitaires mockés
- CI adaptée

fix #13
@Dougniel Dougniel reopened this Nov 7, 2022
Dougniel added a commit that referenced this issue Nov 7, 2022
Erreur `ReferenceError: error is not defined`

fix #13
Dougniel added a commit that referenced this issue Nov 7, 2022
Dougniel added a commit that referenced this issue Nov 7, 2022
Expressions `$expr` non évaluées

fix #13
Dougniel added a commit that referenced this issue Nov 7, 2022
Dougniel added a commit that referenced this issue Nov 7, 2022
@Dougniel
Copy link
Member Author

Dougniel commented Nov 7, 2022

Voilà 12-14s en moins à chaque lancement 💰:

image

Par rapport à avant :
image

Dougniel added a commit that referenced this issue Nov 7, 2022
* refactor: Portage Docker/Python vers js  ⏱

- Expressions `$expr` non évaluées
- Fix url de status

fix #13
Dougniel added a commit that referenced this issue Nov 7, 2022
- Expressions `$expr` non évaluées
- Fix url de status

fix #13
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2022

🎉 This issue has been resolved in version 1.3.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@adriens
Copy link
Member

adriens commented Nov 7, 2022

Voilà 12-14s en moins à chaque lancement 💰:

C'est énorme, ça c'est du ROI 👏

Tu penses que tu aurais quelques RUNs sur différents projets avec des temps pour moi stp ?

@adriens
Copy link
Member

adriens commented Nov 7, 2022

(Je souhaite faire une annonce pour cette release, avec quelques chiffres remarqsuables)

@opt-nc opt-nc deleted a comment from github-actions bot Nov 8, 2022
@Dougniel
Copy link
Member Author

Dougniel commented Nov 8, 2022

Voilà 12-14s en moins à chaque lancement 💰:

C'est énorme, ça c'est du ROI 👏

Tu penses que tu aurais quelques RUNs sur différents projets avec des temps pour moi stp ?

  • Pour le plus avancé c'est proxy-docubase à mon sens car on déploie une image Docker, par contre on est pénalisé sur la génération native : 5m42 (build natif) + 42s de déploiement donc 6m24 au total (en non natif on descendrait à un build de < 50s donc point un total <1m40):

image

  • Pour un cas représentatif c'est Pyramide parce qu'il est fait JHipster, il fait pas au porte monnaie car à lui seul il coûte 14m 💰, en termes d'attente cela représente 7m30 😅 au total (build SB / build Angluar - sans la dernière version de tower-deploy-action) :

image

  • Pour un type d'application consumer (ex: consumer-sig, c'est pas le plus lent) on arrive à un temps de build déploiement à 2m52 au total (ce qui reste important mais sans la dernière version de tower-deploy-action: -14s et sans une gestion du cache des dépendances maven efficace comme ce qui a été fait pour proxy-docubase dernièrement avec une version statique)

image

☝️une partie non négligeable du temps de déploiement est liée à un état d'attente de Tower "pending" (j'avais déjà signalé) avant le lancement, si ce temps pouvait être supprimé on économiserai du temps et de l'argent 💰 (ex: pour un total de déploiement Tower sur l'exemple du proxy-docubase de 42s env 14s sont en pending)

@adriens
Copy link
Member

adriens commented Nov 8, 2022

C'est parfait... maintenant on va coicner sur le temps de download des artefacts (jar, imager docker, ...). j'ai tous les éléments dont j'ai besoin.

est liée à un état d'attente de Tower "pending" (j'avais déjà signalé) avant le lancement,

Oui, tout à fait, je ne l'ai pas oublié 👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continous Integration devops enhancement New feature or request released RUN
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants