search

etalab / transport-ops

Scripts and config files for provision and deploy transport and required services
MIT License
2 stars 2 forks source link

Correction installation tzdata + début d'ajout d'outils de test #32

Closed thbar closed 3 years ago

thbar commented 3 years ago

Deux points dans cette PR :

Correction installation tzdata

tzdata ne pouvait pas être installé car il manquait un apt-get update.

Je déplace l'installation après le update existant (cf #31 pour la remontée d'origine).

Début d'automatisation sur la construction de l'image Docker

Cf #30, les builds Docker via Docker Hub ont été historiquement compliquées (pour partie du fait de la taille de l'organisation, du système de délégation de droits pas suffisamment poussé, et aussi de bugs dans l'UX).

Les travaux précédemment réalisés sur https://github.com/etalab/transport-tools ont permis de défricher le fait de builder et publier une image via GitHub action et en direction du GitHub Container registry, et de voir que ça se passait bien.

Du coup cette PR ajoute une build Docker de l'image, avec 3 tests qui vérifient simplement que node/elixir/erlang sont installés et à la bonne version (uniquement version majeure pour Erlang), de façon à bloquer une PR ou une build.

Comment cela fonctionne : une image de "test" est buildée en local, dans le container action, puis des commandes sont lancées dessus.

Aucune publication n'est faite à ce stade, mais j'ai mis des notes pour y aller après.

Il faudra réfléchir au workflow souhaité.

Ce travail va aider à 1/ éviter les erreurs en local spécifiques à un développeur (encore la semaine dernière) et 2/ automatiser pour disposer de patchs sécurité plus fréquents.

thbar commented 3 years ago

@AntoineAugusti merci pour les retours, peux-tu re-reviewer ? J'ai répondu aux questions, et traité la remarque sur le env. Merci !

thbar commented 3 years ago

Désolé de t'avoir causé du tort avec GitHub Actions. Ça me parait bon, j'ai aussi jeté un oeil à l'output du job.

@AntoineAugusti haha pas de mal :-) mais merci pour ta compassion appréciée. Ok top !