Merge pull request #425 from mbaldessari/fix-uid

Fixes and cleanups

Author: mbaldessari

scripts/pattern-util.sh

@@ -1,9 +1,15 @@
 #!/bin/bash
+function is_available {
+  command -v $1 >/dev/null 2>&1 || { echo >&2 "$1 is required but it's not installed. Aborting."; exit 1; }
+}
 
 if [ -z "$PATTERN_UTILITY_CONTAINER" ]; then
 	PATTERN_UTILITY_CONTAINER="quay.io/hybridcloudpatterns/utility-container"
 fi
 
+readonly commands=(podman)
+for cmd in ${commands[@]}; do is_available "$cmd"; done
+
 UNSUPPORTED_PODMAN_VERSIONS="1.6 1.5"
 for i in ${UNSUPPORTED_PODMAN_VERSIONS}; do
 	# We add a space
@@ -25,15 +31,17 @@ fi
 # $HOME is mounted as itself for any files that are referenced with absolute paths
 # $HOME is mounted to /root because the UID in the container is 0 and that's where SSH looks for credentials
 
-# Do not quote the ${KUBECONF_ENV} below, otherwise we will pass '' to podman
-# which will be confused
+# We do not rely on bash's $UID and $GID because on MacOSX $GID is not set
+MYUID=$(id -u)
+MYGID=$(id -g)
 podman run -it --rm --pull=newer \
 	--security-opt label=disable \
+	--user "${MYUID}:${MYGID}" \
+	--userns "keep-id:uid=${MYUID},gid=${MYGID}" \
 	-e EXTRA_HELM_OPTS \
 	-e KUBECONFIG \
 	-v "${HOME}":"${HOME}" \
 	-v "${HOME}":/pattern-home \
-	-v "${HOME}":/root \
 	-w "$(pwd)" \
 	"$PATTERN_UTILITY_CONTAINER" \
 	$@