Skip to content

improve Dockerfile build sequence#3274

Open
pifou25 wants to merge 2 commits into
jeedom:developfrom
pifou25:feat/dockerfile_copy_develop
Open

improve Dockerfile build sequence#3274
pifou25 wants to merge 2 commits into
jeedom:developfrom
pifou25:feat/dockerfile_copy_develop

Conversation

@pifou25

@pifou25 pifou25 commented Apr 19, 2026

Copy link
Copy Markdown
Contributor

Description

Pour éviter de recopier les sources lorsque ce n'est pas nécessaire, au démarrage du conteneur: le step (6) dans Dockerfile COPY . ${WEBSERVER_HOME} fait déjà la copie des fichiers dans le répertoire courant.
Pourtant je l'ai quand même laissé post-install dans le init.sh pour un cas particulier :

docker run -p 80:80 -v /home/pifou/jeedom:/var/www/html --rm --name jeedom_server jeedom
Avec l'option -v je map le répertoire de l'host sur celui du container. Du coup ça permet de persister Jeedom sur l'host, et ça marche lors de l'arrêt / mise à jour / relance du container, on ne perd rien. Mais au 1er lancement, si mon rep /home/pifou/jeedom est vide sur l'host, il est vide aussi dans le container, malgré que je pensais y avoir copié les sources (lors du build). Dans ce cas uniquement, on doit recopier les sources au démarrage du conteneur, cela est normal.

Dans le cas de l'utilisation d'un volume nommé, c'est différent, le répertoire n'existe pas et il est bien initialisé avec les données de l'image du conteneur (le volume persistant sera créé dans /var/lib/docker) :
docker run -p 80:80 -v jeedom_data:/var/www/html --rm --name jeedom_server jeedom

Le fichier .dockerignore permet à l'instruction COPY . de savoir ce qu'il faut ignorer - ne pas copier dans l'image.

Le step 10 (install composer) se fait aussi au build de l'image, pas au démarrage de l'application. Donc dans le Dockerfile également.

Suggested changelog entry

  • Ne pas écraser notre répertoire source Jeedom lors d'une mise à jour de l'image Docker.

@pifou25 pifou25 mentioned this pull request Apr 19, 2026
11 tasks
@pifou25 pifou25 closed this Apr 19, 2026
@pifou25 pifou25 reopened this Apr 19, 2026
Comment thread install/OS_specific/Docker/init.sh Outdated
Comment thread install/OS_specific/Docker/init.sh Outdated
@pifou25 pifou25 force-pushed the feat/dockerfile_copy_develop branch 2 times, most recently from abf1df2 to be05783 Compare June 13, 2026 20:23
Co-Authored-By: kwizer15 <kwizer15@users.noreply.github.com>
@pifou25 pifou25 force-pushed the feat/dockerfile_copy_develop branch from be05783 to d8ae0c0 Compare June 14, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants