From 9816fe18e839eb195a6f4482311b65df995b920f Mon Sep 17 00:00:00 2001 From: Meghana Madhusudhan Date: Tue, 26 Aug 2014 16:40:49 +0530 Subject: Hook scripts should exit with correct exit status. When volume set options other than nfs-ganesha.host and nfs-ganesha.enable are executed, the hook script S31ganesha-set.sh should exit with exit status 0. The present behaviour results in an error message in one of the logs that could be misleading. Similarly, hook script S30samba-set.sh should exit with exit status 0 if samba is not installed. Fixing the issue with a minor change in the script. Change-Id: I35d1dc8400bb7043385800ed97adc5321829437f BUG: 1122978 Signed-off-by: Meghana Madhusudhan Reviewed-on: http://review.gluster.org/8385 Tested-by: Gluster Build System Reviewed-by: Vijay Bellur --- extras/hook-scripts/set/post/S31ganesha-set.sh | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) (limited to 'extras/hook-scripts/set/post/S31ganesha-set.sh') diff --git a/extras/hook-scripts/set/post/S31ganesha-set.sh b/extras/hook-scripts/set/post/S31ganesha-set.sh index b09a3942a8e..f839aa37061 100755 --- a/extras/hook-scripts/set/post/S31ganesha-set.sh +++ b/extras/hook-scripts/set/post/S31ganesha-set.sh @@ -3,6 +3,7 @@ PROGNAME="Sganesha-set" OPTSPEC="volname:,gd-workdir:" VOL= declare -i EXPORT_ID +ganesha_key="FALSE" GANESHA_DIR="/var/lib/glusterfs-ganesha" CONF1="$GANESHA_DIR/nfs-ganesha.conf" GANESHA_LOG_DIR="/var/log/nfs-ganesha/" @@ -13,7 +14,6 @@ host_name="none" LOC="" GLUSTERD_WORKDIR= - function parse_args () { ARGS=$(getopt -l $OPTSPEC -o "o" -name $PROGNAME $@) @@ -34,12 +34,14 @@ function parse_args () for pair in $@; do read key value < <(echo "$pair" | tr "=" " ") case "$key" in - "nfs-ganesha.enable") - enable_ganesha=$value - ;; - "nfs-ganesha.host") - host_name=$value - ;; + "nfs-ganesha.enable") + enable_ganesha=$value + ganesha_key="TRUE" + ;; + "nfs-ganesha.host") + host_name=$value + ganesha_key="TRUE" + ;; *) ;; esac @@ -257,6 +259,10 @@ function stop_ganesha() } parse_args $@ + if [ "$ganesha_key" == "FALSE" ] + then + exit 0 + fi check_ganesha_dir $VOL if echo $enable_ganesha | grep -q -i "ON" then -- cgit