search

hepsw / docks

a set of Dockerfiles defining docker containers for HEP software and appliances.
BSD 3-Clause "New" or "Revised" License
17 stars 13 forks source link

Replacing cubied with run-cvmfs.sh (issue #20) #21

Open kreczko opened 7 years ago

kreczko commented 7 years ago

PR regarding issue #20

Tested with

docker build -t hepsw/cvmfs-base .
docker run --privileged -ti hepsw/cvmfs-base ls -l /cvmfs/sft.cern.ch/lcg/
sbinet commented 7 years ago

ah. well, cubied is still needed (so it can properly act as a mini PID 1 process, reaping zombies and such).

wouldn't it work with adding another shell script (so it doesn't confuse gitlab-runner) that runs cubied ?

kreczko commented 7 years ago

At the moment everything you run in the container is encapsulated in cubied, e.g.

#!/usr/bin/cubied

echo "::: cvmfs-config..."
cvmfs_config setup || exit 1

echo "::: mounting FUSE..."
mount -a
echo "::: mounting FUSE... [done]"

exec "$@"

This includes the gitlab-runner shell check: https://gitlab.com/gitlab-org/gitlab-ci-multi-runner/blob/master/shells/bash.go

Unless the gitlab-runner is able to resolve cubied, I don't think there is a better workaround.

That said, our CI container (cvmfs-base →cvmfs-lz →lz-builder) is independent of this PR, so there is no need to merge until we find a better solution.