Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Pour éviter de recopier les sources lorsque ce n'est pas nécessaire: 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 /tmp/jeedom:/var/www/html --rm --name jeedom_server jeedom
Avec l'option -v je map le rep 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 / relance du container, on ne perd rien. Mais au 1er lancement, si mon rep /tmp/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.
C'est ce que je ne comprends pas dans la doc, normalement si le rep est vide on devrait y retrouver les données du container:
https://docs.docker.com/engine/storage/volumes/#mounting-a-volume-over-existing-data
Le fichier
.dockerignore
permet à l'instructionCOPY .
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.
J'ai mis à jour
composer.lock
de nouveau mais c'est normalement le même que celui en attente de merge sur #2905 mais le build ne marche pas avec lecomposer.lock
actuel (c'est sans doute pour ça qu'il est systématiquement supprimé)Suggested changelog entry
Related issues/external references
Fixes #
Types of changes
PR checklist