pix
pix copied to clipboard
[TECH] Remise à jour de la config `docker compose`
:unicorn: Problème
Lors du docker compose up -d
nécessaire pour lancer l'environnement de développement, un warning est lancé :
WARN[0000] /pix/docker-compose.yaml: `version` is obsolete
:robot: Proposition
- Retirer le champ
version
déprécié, - Utiliser le nom de fichier
compose.yaml
qui semble être le nom de fichier de config recommandé par Docker.
:rainbow: Remarques
J'ai laissé en yaml
plutôt qu'en yml
(je ne sais pas ce que ça change).
:100: Pour tester
Vérifier que le log de warning disparait à l'usage de docker compose
.
Je n'ai pas ce warning au lancement de la commande perso.
Je n'ai pas ce warning au lancement de la commande perso.
Intéressant 🤔 je suis en Docker version 26.0.0, build 2ae903e
, tu peux me donner la tienne ?
Je n'ai pas ce warning au lancement de la commande perso.
Intéressant 🤔 je suis en
Docker version 26.0.0, build 2ae903e
, tu peux me donner la tienne ?
Ca doit jouer, je suis en 25.0.3, build 4debf41
EDIT: Effectivement, j'ai le warning après avoir mis à jour ma version de Docker.
@yannbertrand Merci pour la PR !
Est-ce qu'il ne faudrait pas aussi faire les modifications sur les autres fichiers docker-compose dans le projet ?
Je pense notamment à ceux :
- Dans le dossier docker
- Dans les tests e2e
- Dans le dossier local-domains pour les SSO qu'on utilise côté Accès 😄
@yannbertrand Merci pour la PR ! Est-ce qu'il ne faudrait pas aussi faire les modifications sur les autres fichiers docker-compose dans le projet ? Je pense notamment à ceux :
* Dans le dossier [docker](https://github.com/1024pix/pix/tree/bump-docker-compose-config-file/docker) * Dans les tests [e2e](https://github.com/1024pix/pix/blob/bump-docker-compose-config-file/high-level-tests/e2e/docker-compose.yaml) * Dans le dossier [local-domains](https://github.com/1024pix/pix/blob/bump-docker-compose-config-file/scripts/local-domains/docker-compose.yaml) pour les SSO qu'on utilise côté Accès 😄
@er-lim je ne suis pas utilisateur de ces fichiers, je vous laisse gérer dans une autre PR pour ce qui concerne le scope accès ? Au cas où je casse quelque chose dont je ne connais pas l'existence 😬