Permalink
Browse files

take stronger action in launcher when people are using an unsupported…

… filesystem
1 parent 2884e5f commit 48f22d14f39496c8df446cbc65ee04b258c5a1a0 @SamSaffron SamSaffron committed May 29, 2015
Showing with 7 additions and 9 deletions.
  1. +7 −9 launcher
View
@@ -102,16 +102,14 @@ prereqs() {
# 2. running aufs or btrfs
test=`$docker_path info 2> /dev/null | grep 'Driver: '`
- if [[ "$test" =~ [aufs|btrfs] ]] ; then : ; else
- echo "Your Docker installation is not using the recommended AuFS (union filesystem) and may be unstable."
- echo "If you are unable to bootstrap / stop your image please report the issue at:"
- echo "https://meta.discourse.org/t/discourse-docker-installation-without-aufs/15639"
+ if [[ "$test" =~ [aufs|btrfs|zfs|overlay] ]] ; then : ; else
+ echo "Your Docker installation is not using a supported filesystem if we were to proceed you may have a broken install."
+ echo "aufs is the recommended filesystem you should be using (zfs/btrfs and overlay may work as well)"
+ echo "You can tell what filesystem you are using by running \"docker info\" and looking at the driver"
echo ""
- read -p "Continue without proper filesystem? [yN]" yn
- case $yn in
- [Yy]* ) break;;
- * ) exit 1;;
- esac
+ echo "If you wish to continue anyway using your existing unsupported filesystem"
+ echo "read the source code of launcher and figure out how to bypass this."
+ exit 1
fi
# 3. running recommended docker version

2 comments on commit 48f22d1

@hyperknot
hyperknot commented on 48f22d1 Nov 14, 2016 edited

[aufs|btrfs|zfs|overlay] actually matches a single character in the list aufs|btrzovely.

My idea would be to use something like this:

test=`$docker_path info 2> /dev/null | grep 'Storage Driver: '`
if [[ "$test" =~ "Storage\ Driver:\ (aufs|btrfs|zfs|overlay)" ]]

Yet, this is somehow not maching in bash, only in pcre. Might need to use grep -e for this.

Without grep storage driver it'd output 3 lines:

Storage Driver: devicemapper
Logging Driver: json-file
Cgroup Driver: cgroupfs
Please sign in to comment.