From f48656b32e328f96bda5aade55070867d5faf333 Mon Sep 17 00:00:00 2001 From: Raghavendra Talur Date: Tue, 29 Apr 2014 14:28:29 +0530 Subject: extras/hookscripts: Remove %P from smb share path value. We had added %P in the value for path in smb share definitions to avoid ctdb errors. However, %P does not work well for windows clients and they fail to access the share. Solution for the original problem: The error messages were being generated by a monitoring script that ctdb runs. The script reads the path value from testparm and tests if it exists in the local machine. This particular check does not hold valid for virtual file system backends like gluster. There is no harm in asking ctdb to not to check the existence of such backend and can be done by setting the following config in ctdb.conf: CTDB_SAMBA_SKIP_SHARE_CHECK=YES. Change-Id: I91627c8a279126a5cc24c6347847f9bfbad53dd5 BUG: 1068776 Signed-off-by: Raghavendra Talur Reviewed-on: http://review.gluster.org/7598 Reviewed-by: Jose A. Rivera Reviewed-by: Poornima G Tested-by: Gluster Build System Reviewed-by: Harshavardhana --- extras/hook-scripts/start/post/S30samba-start.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'extras/hook-scripts/start/post/S30samba-start.sh') diff --git a/extras/hook-scripts/start/post/S30samba-start.sh b/extras/hook-scripts/start/post/S30samba-start.sh index 529348e25c4..34fde0ef8c0 100755 --- a/extras/hook-scripts/start/post/S30samba-start.sh +++ b/extras/hook-scripts/start/post/S30samba-start.sh @@ -65,7 +65,7 @@ function add_samba_share () { STRING+="glusterfs:volume = $volname\n" STRING+="glusterfs:logfile = $LOGFILEBASE/glusterfs-$volname.%%M.log\n" STRING+="glusterfs:loglevel = 7\n" - STRING+="path = %%P/\n" + STRING+="path = /\n" STRING+="read only = no\n" STRING+="guest ok = yes\n" printf "$STRING" >> ${CONFIGFILE} -- cgit