[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