Automatically build Docker images

added 1 commit

• 0b042779 - Fix syntax

Compare with previous version

added 1 commit

• c1ca54f3 - Fix syntax

Compare with previous version

added 1 commit

• 48bb4d05 - Change kaniko's tag

Compare with previous version

added 1 commit

• 6cdcead9 - Use OTB's archive repository

Compare with previous version

added 1 commit

• ab80719a - Fix typo

Compare with previous version

added 3 commits

• 34febcf3 - Automatically build Docker images
• 807be5af - Change kaniko's tag
• 38fa81f0 - Use OTB's archive repository

Compare with previous version

added 1 commit

• b6277f92 - Push images into Gitalb's registry

Compare with previous version

@lhermitte as you can see, the pipeline published all images into the internal registry: https://gitlab.orfeo-toolbox.org/s1-tiling/s1tiling-dockers/container_registry/82

The only trick: use an ARG to define the prefix of the image's name. This solution allows the user to build the images locally, using his local cache.

In the commit I did not pushed (at the time), I had already changed this part of the dockerfile to

ARG OTB_VERSION=7.2.0
ARG REPOSITORY=registry.orfeo-toolbox.org/s1-tiling/s1tiling-dockers
FROM ${REPOSITORY}:ubuntu-otb${OTB_VERSION}-base

But now I'm not sure what's the correct way of naming docker images.

added 2 commits

• 46e62e0a - 1 commit from branch master
• 5b367b08 - Merge branch 'master' into add-ci

Compare with previous version

IMHO you missed something in the default REPOSITORY: the name of the image.

Docker defines:

• a registry (registry.orfeo-toolbox.org)
• an image's name, with /
• a tag (after :)

In Gitlab, the path is fixed: <group>/<project>/<name>

The most perturbing part of the Docker's naming scheme is the registry is also fully part of the name of the image. But as an image can have many names... like a memory address can be stored in many variables ;-)

added 1 commit

• c1c56685 - Change Docker images naming scheme

Compare with previous version

added 1 commit

• 97c1c40a - Paralelize jobs

Compare with previous version

@lhermitte ça commence à fonctionner. J'ai fait les modifs pour le naming scheme, y compris dans la doc. J'espère que j'ai juste.

Sur le dernier commit, je me suis fait plaisir et j'ai retravaillé le pipeline pour paraléliser tout en factorisant les parties utiles. J'espère que ça reste compréhensible.

approved this merge request

marked this merge request as ready

@gbonnefille Cela m'a l'air pas mal oui. C'est iso-fonctionnel coté script -- il me produit la meme image que la dernière en 7.3 que j'avais générée et avec le bon nouveau nom.

Je regarde la modif sur la doc et j'intègre tout ça.

Merci énormément dans tous les cas!

mentioned in commit abd80f10

merged