Skip to content

dépendance de build (compose >= v2.36.2) - #110

Open
bouttier wants to merge 7 commits into
PnX-SI:mainfrom
bouttier:feat/compose_v2_36_2
Open

dépendance de build (compose >= v2.36.2)#110
bouttier wants to merge 7 commits into
PnX-SI:mainfrom
bouttier:feat/compose_v2_36_2

Conversation

@bouttier

Copy link
Copy Markdown
Contributor

(voir juste le dernier commit)
Ajout des dépendances de build via additional_contexts. Ceci permet de build via docker compose build et sans nécessiter le script build.sh.

Cette fonctionnalité est correctement supportée par docker compose >= v2.36.2. Pour rester compatible avec la version de compose fournit dans les dépôts de Debian 13, les additional_contexts ont été placé dans docker-compose-build-deps.yml.

À noter qu’il est facile d’avoir un compose à jour en rajoutant les dépôts de docker à ses sources apt, c’est même ce qui est préconisé dans le guide d’installation de docker : https://docs.docker.com/engine/install/debian/

As api & worker depends on install-db, the image is necessary built.
buildx run images build in parallel, depends_on does not solve this
issue as it add run dependency.
build section added to userhub directly, so it becomes possible to do:

  docker compose up -d --build userhub
Best pratice for production are to build images on a CI and push them on
a registry, but we may have some users requiring a build with a specific
module and not able to do the CI/registry stuff.

Simplify build.sh using docker compose build directly.
@bouttier
bouttier force-pushed the feat/compose_v2_36_2 branch from 1558682 to 5f58b02 Compare April 28, 2026 19:08
Requires docker compose >= 2.36.2
@christophe-ramet

christophe-ramet commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

J'avais commenté dans cette PR : #109 (comment)

Pour ce qui de #110, je trouve ça super comme fonctionnalité, mais pour moi c'est pas une bonne idée de l'intégrer tant que la plupart des gens n'ont pas une version de compose compatible.

Mais après réflexion, je pense que mon argument est mauvais. Mon idée de départ était d'attendre que cette version de compose soit dans toutes les versions de Debian que l'on supporte. Si on fait ça, on va devoir attendre la fin de vie de BookWorm (Juin 2028).
Par ailleurs, la méthode officielle d'installation de docker passe par l'usage du repo de Docker et non pas par les repos officiels de Debian.

Cette version de compose depuis plus d'un an, je te rejoins @bouttier, ce serait bien qu'on ajoute cette fonctionnalité.

Par ailleurs, seules les personnes désirant build elles-mêmes les images auront besoin d'une version de compose à jour, pas l'utilisateur moyen de GDS.

Qu'est ce que tu en pense @jacquesfize ?

@jacquesfize

Copy link
Copy Markdown
Contributor

Je suis d'accord ! Je rajouterais peut-être par sécurité une ligne dans le README indiquant la version minimal de Docker supportée.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants