glusterfs.git/extras/hook-scripts/start/post/S29CTDBsetup.sh, branch v3.12.5

hook-scripts: S29CTDBsetup: clean and fix mount commands and options

2016-01-15T13:52:56+00:00

This refactors the mount options of S29CTDBstart.sh into one
variable, removing the superfluous mentioning of 'default'.
The manual mount command is fixed to:
- also use the common options, thereby adding
  _netdev,transport=tcp
  (while this does not have an effect for the _netdev
   option, it makes a difference for the transport option),
- also use the HOSTNAME variable just like the fstab entry,
- use a "/" in the remote location specification.
Hence it now behaves identically to the fstab mount.

Change-Id: Ibb7613b1b1278ab13745846baa79268db226ef19
BUG: 1295520
Signed-off-by: Michael Adam 
Reviewed-on: http://review.gluster.org/13170
Tested-by: Raghavendra Talur 
Reviewed-by: Raghavendra Talur 
Reviewed-by: Rajesh Joseph 
NetBSD-regression: NetBSD Build System 
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Niels de Vos

hook-scripts: remove RHS-2.0 reference from S29CTDB scripts

2016-01-11T09:09:38+00:00

This is old. And such downstream reference should not
appear in upstream code.

Change-Id: Ifa0664dae0fe5ac913a49afe2fefa43b0112b024
BUG: 1295505
Signed-off-by: Michael Adam 
Reviewed-on: http://review.gluster.org/13169
Tested-by: Gluster Build System 
Reviewed-by: Anoop C S 
Reviewed-by: Raghavendra Talur

hook-scripts: remove outdated comment from C29CTDB start scipt

2016-01-10T10:52:54+00:00

This script does not change Samba's config any more.

Change-Id: Ie6001f9a49006f95b291e24252dc362f2a7db14c
BUG: 1295504
Signed-off-by: Michael Adam 
Reviewed-on: http://review.gluster.org/13168
Tested-by: Gluster Build System 
Reviewed-by: Anoop C S 
Reviewed-by: Niels de Vos

hook-scripts: don't let ctdb script change samba config

2015-12-16T19:13:36+00:00

There are several reasons why the behaviour in
the hook scripts was bad:

1. A samba installation is clustered or non-clustered.
   That does not change because of the availability
   of the CTDB lock-volume. If the lock-volume is not
   available (and hence CTDB is not available or not
   healthy), then Samba won't be operational. But turning
   it into a non-clustered Samba-installation can in
   the worst case lead to data corruption if clients
   manage to access the same files (on share volumes).
   Hence 'clustering = yes/no' in Samba's config should
   not be touched.

   In particular, Samba should not be stopped/started by
   the hook script. If needed, then ctdb will take care
   of it.

2. Changing the idmap configuration is potentially
   dangerous as well. In particular the used tdb2
   backend is legacy nowadays and should not be used
   any more in new installs. (I stems from the times
   when ctdb could not host persistent databases.)
   Changing the idmap can result in loss of access
   to files or in giving access to files where it is
   not intended.

3. The pattern used for detecting need for change is
   fragile. It may or may not play well possible
   manual changes to smb.conf.

This change removes the parts that change the smb.conf
file and start or stop Samba from the S29CTDB* hook scripts.

Change-Id: I72f7aabafa8f089da4531fca2572a72c22825bcc
BUG: 1290151
Signed-off-by: Michael Adam 
Reviewed-on: http://review.gluster.org/12930
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Talur

scripts: CTDB lockvol ping-timeout should be 10secs

2014-06-12T06:08:36+00:00

commit "4044e387dd40e2d12145fd32d5fd715406fcec8e" approach
to this led to deadlocks instead we rely on an old trick at
client side - without any need to modifying the volume globally.

Change-Id: I6e3c6bf9c0a3a0dd99a51b5d2a6c1f050ed2384c
BUG: 1091600
Signed-off-by: Harshavardhana 
Reviewed-on: http://review.gluster.org/8007
Reviewed-by: Raghavendra Talur 
Reviewed-by: Poornima G 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

scripts: CTDB lockvol ping-timeout should be 10secs

2014-05-02T15:52:40+00:00

Users often forget to update the ping-timeout, perhaps
we should take care of this automatically

Change-Id: I6d14df8aa52665847233cb1760dfd54f41baadec
BUG: 1091600
Signed-off-by: Harshavardhana 
Reviewed-on: http://review.gluster.org/7569
Reviewed-by: Raghavendra Talur 
Reviewed-by: Niels de Vos 
Reviewed-by: Jose A. Rivera 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

scripts: CTDB hooks should add `_netdev` for network mounts

2014-03-12T06:45:14+00:00

Change-Id: I86ad5e9ed291866a8d478768fb447e269f51c39e
BUG: 1075182
Signed-off-by: Harshavardhana 
Reviewed-on: http://review.gluster.org/7221
Reviewed-by: Niels de Vos 
Reviewed-by: Jose A. Rivera 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

hook-scripts: Move smb hooks to right place.

2014-03-10T13:38:21+00:00

Moved the hook-scripts to dirs corresponding to event.
Also, edited S30samba-set.sh to with changes in
other two hook scripts to bring uniformity.

Change-Id: Ib98c0a18d4bb5d0ba4b881b2e19b40d72580b1c2
BUG: 1073468
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/7201
Reviewed-by: Poornima G 
Reviewed-by: Bala FA 
Tested-by: Gluster Build System