skilld-labs / skilld-docker-container

Starterkit for drupal development
MIT License
20 stars 32 forks source link

Fixing nasty OSX bug where docker mount root dir #409

Closed davidferlay closed 2 years ago

davidferlay commented 2 years ago

Before :

/bin/sh: realpath: command not found
make -s down
Removing network & containers for skilld
[+] Running 4/4
 ⠿ Container skilld_web   Removed                                                                                                                                0.4s
 ⠿ Container skilld_mail  Removed                                                                                                                                0.3s
 ⠿ Container skilld_php   Removed                                                                                                                                0.4s
 ⠿ Network skilld_front   Removed                                                                                                                                0.1s
CURDIR=/Users/skilld/Sources/ski-platform/roles/techs/services/skilld-landing/files
#
#@docker run --rm --user 0:0 -v /Users/skilld/Sources/ski-platform/roles/techs/services/skilld-landing/files:/mnt -w /mnt -e RMLIST=" web/core web/libraries web/modules/contrib web/profiles/contrib web/sites web/themes/contrib vendor" skilldlabs/php:74-fpm sh -c 'for i in $RMLIST; do rm -fr $i && echo "Removed $i"; done'
Clean-up database data from / ...
docker run --rm --user 0:0 -v /:/mnt skilldlabs/php:74-fpm sh -c "rm -fr /mnt/`basename /`"
rm: can't remove '/mnt/var/run/linuxkit-external-logging.sock': Resource busy
rm: can't remove '/mnt/var/run/linuxkit-containerd/containerd.sock': Resource busy
rm: can't remove '/mnt/var/log': Resource busy

After :

CURDIR=/Users/skilld/Sources/ski-platform/roles/techs/services/skilld-landing/files
#@docker run --rm --user 0:0 -v /Users/skilld/Sources/ski-platform/roles/techs/services/skilld-landing/files:/mnt -w /mnt -e RMLIST=" web/core web/libraries web/modules/contrib web/profiles/contrib web/sites web/themes/contrib vendor" skilldlabs/php:74-fpm sh -c 'for i in $RMLIST; do rm -fr $i && echo "Removed $i"; done'
Clean-up database data from /Users/skilld/Sources/ski-platform/roles/techs/services/skilld-landing/files/.cache ...
docker run --rm --user 0:0 -v /Users/skilld/Sources/ski-platform/roles/techs/services/skilld-landing/files:/mnt skilldlabs/php:74-fpm sh -c "rm -fr /mnt/`basename /Users/skilld/Sources/ski-platform/roles/techs/services/skilld-landing/files/.cache`"
andypost commented 2 years ago

Not clear why PHP_VERSION=7 for osx added

davidferlay commented 2 years ago

Well that's actually a good question @andypost

Maybe an OSX user can enlighten us ?

Edit: Found back https://github.com/skilld-labs/skilld-docker-container/issues/400 > Need to check if it is exhaustive and accurate

andypost commented 2 years ago

I bet it remains from past when we used to switch to php 7 from 5

artreaktor-niks commented 2 years ago

the main part in env extra code is CUID=1000 CGID=1000

But for the PHP_VERSION=7 actually not sure. This by default goes from project to project. I think it could be removed.