[arvados] updated: 2.1.0-2649-g9ae866579
git repository hosting
git at public.arvados.org
Thu Jun 30 17:22:04 UTC 2022
Summary of changes:
tools/salt-install/installer.sh | 6 ++++++
1 file changed, 6 insertions(+)
via 9ae866579b9e62ac11817b8053aaae74be7ea1cf (commit)
from b09d1b51bbaa0a7709166187bb007683053205ed (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit 9ae866579b9e62ac11817b8053aaae74be7ea1cf
Author: Peter Amstutz <peter.amstutz at curii.com>
Date: Thu Jun 30 13:21:26 2022 -0400
18870: Handle possibly undefined command line parameters
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>
diff --git a/tools/salt-install/installer.sh b/tools/salt-install/installer.sh
index 238efe868..35a194af4 100755
--- a/tools/salt-install/installer.sh
+++ b/tools/salt-install/installer.sh
@@ -116,9 +116,11 @@ case "$subcmd" in
exit
fi
+ set +u
SETUPDIR=$1
PARAMS=$2
SLS=$3
+ set -u
err=
if [[ -z "$PARAMS" || ! -f local.params.example.$PARAMS ]] ; then
@@ -156,7 +158,9 @@ case "$subcmd" in
echo "setup directory initialized, now go to $SETUPDIR, edit '${CONFIG_FILE}' and '${CONFIG_DIR}' as needed, then run 'installer.sh deploy'"
;;
deploy)
+ set +u
NODE=$1
+ set -u
loadconfig
@@ -218,7 +222,9 @@ case "$subcmd" in
diagnostics)
loadconfig
+ set +u
declare LOCATION=$1
+ set -u
if ! which arvados-client ; then
echo "arvados-client not found, install 'arvados-client' package with 'apt-get' or 'yum'"
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list