pix icon indicating copy to clipboard operation
pix copied to clipboard

[TECH] Remise à jour de la config `docker compose`

Open yannbertrand opened this issue 10 months ago • 5 comments

: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

: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.

yannbertrand avatar Apr 18 '24 13:04 yannbertrand

Je n'ai pas ce warning au lancement de la commande perso. Capture d’écran 2024-04-19 à 08 44 26

alexandrecoin avatar Apr 19 '24 06:04 alexandrecoin

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 ?

yannbertrand avatar Apr 19 '24 07:04 yannbertrand

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.

alexandrecoin avatar Apr 19 '24 07:04 alexandrecoin

@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 😄

er-lim avatar Apr 22 '24 07:04 er-lim

@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 😬

yannbertrand avatar Apr 22 '24 07:04 yannbertrand