glusterfs.git/extras/ganesha/scripts/ganesha-ha.sh, branch v3.7.4

common-ha : refresh-config should print sensible output

2015-08-25T03:47:09+00:00

ganesha-ha.sh --refresh-config printed the dbus-send output
messages as is on the console. Improving the output
of the operation by redirecting the messages to /var/log/messages
and by checking the exit code of the command executed.

The behaviour is also changed a litlle by exiting
when refresh-config fails on any of the nodes.
We don't want to continue changing config files
on other nodes when refresh-config has already failed
on one of the nodes.

Change-Id: Ic2e4e26bc09b1ba23cc6b62a1590c6083a0bde80
BUG: 1254494
Signed-off-by: Meghana M 
Reviewed-on: http://review.gluster.org/11949
Reviewed-by: soumya k 
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Tested-by: NetBSD Build System

common-ha: concise output for HA status

2015-08-24T05:55:55+00:00

filter out unnecessary detail from the HA status

Change-Id: I407fbec1c61aa35afa7a433187a122975bf3e04d
BUG: 1250628
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/11942
Tested-by: NetBSD Build System 
Reviewed-by: soumya k 
Tested-by: Gluster Build System 
Reviewed-by: Meghana M

common-ha: delete virt-IP entry of deleted node

2015-08-21T11:55:34+00:00

Avoid accumulating invalid/defunct virtual IP entries in the HA
config file.

use correct 'clean' name when adding a node and its virtual IP

Also fix a nit in the sample HA config file

Change-Id: I2b7ca279f6390ee34b4e0689039eb7d846ed7e34
BUG: 1250601
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/11841
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Meghana M 
Reviewed-by: soumya k

NFS-Ganesha : Add-node does not copy "exports" directory correctly

2015-07-15T00:56:34+00:00

Add-node logic has to copy the "exports" directory into the
new node in the same path. There was an error in copying to the correct
path. Fixing it.

This is a backport of the patch merged upstream,
http://review.gluster.org/#/c/11618/

Change-Id: I76d8679966372d6fcee2dcf38bc20c9b5c522132
BUG: 1242192
Signed-off-by: Meghana M 
Reviewed-on: http://review.gluster.org/11635
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY

NFS-Ganesha : Export fails on RHEL 7.1

2015-07-10T17:41:17+00:00

We grep for CONFFILE parameter in "/etc/syconfig/ganesha"
file to find out the path of the ganesha config file.
In RHEL 7.1, this parameter does not exist in the file and
we can't find out the ganesha config file. Export
fails invariably due to this. Changing this pattern to
a more generic one and default it to "/etc/ganesha/ganesha.conf"

This patch is a backport of the fix merged upstream,
http://review.gluster.org/#/c/11594/

Change-Id: Ia0f87f964a5771b97d9077f17a0387d7a01e02b6
BUG: 1241885
Signed-off-by: Meghana Madhusudhan 
Reviewed-on: http://review.gluster.org/11621
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY

common-ha: ganesha-ha.sh status tries to read ganesha-ha.conf

2015-07-09T15:17:09+00:00

status doesn't need to read the config

backport of
> Change-Id: Id02252abe52820dbc263f4a880bde72a23b121bd
> BUG: 1241133
> http://review.gluster.org/#/c/11581/

Change-Id: I036fd3072f38958373b1a85b0dc5c97245a72af4
BUG: 1241134
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/11582
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System

common-ha: Fix '/var/lib/nfs/statd/state' path creation

2015-07-01T10:04:41+00:00

'/var/lib/nfs/statd/state' which contains NSM state number
should be a file instead of directory.

This is a backport of the below fix -
http://review.gluster.org/#/c/11468/

Change-Id: Id008b4f4dd810fe6d6b4d2599cbc0b488010384b
BUG: 1238057
Signed-off-by: Soumya Koduri 
Reviewed-on: http://review.gluster.org/11468
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Niels de Vos 
Reviewed-on: http://review.gluster.org/11476
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System

common-ha: fix delete-node

2015-06-26T19:36:09+00:00

delete-node is a designed to be "disruptive"

surgically delete a node from the config and stop nfs-ganesh on that
node.

finish the implementation and fix a few minor issues

>-Id: I964bb72a76ee635b5fc484ec5b541e69eeececcd
> BUG: 1234474
> http://review.gluster.org/11353

Change-Id: Ic06952df8f42f80cfeb82d419757a8b99dff275a
BUG: 1234584
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/11427
Tested-by: Gluster Build System

NFS-Ganesha : Implement refresh-config

2015-06-25T01:08:39+00:00

It is important that we give an automatic way of refreshing the config
when the user has changed the export file manually.
Without this, the user will be forced to restart the server.
Implementing refresh_config by using dbus signals to unexport
and export the same export dynamically.
Making a few changes to make sure that "--help"
doesn't throw unnecessary error messages.

This is the backport of the patch merged on master,
http://review.gluster.org/#/c/11331/

Change-Id: I2e17edeffc191e2cc30a328b2999e1a04f5bc498
BUG: 1235258
Signed-off-by: Meghana M 
Reviewed-on: http://review.gluster.org/11380
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Kaleb KEITHLEY

common-ha : Fixing add node operation

2015-06-22T09:37:11+00:00

Resource create for the added node referenced a variable
new_node that was never passed. This led to a wrong schema
type in the cib file and hence the added node always ended
up in failed state. And also, resources were wrongly
created twice and led to more errors. I have fixed the variable
name and deleted the repetitive invocation of the recreate-resource
function.

The new node has to be added to the existing ganesha-ha config
file for correct behaviour during subsequent add-node operations.
This edited file has to be copied to all the other cluster nodes.
I have added a fix for this as well.

This is a backport of the fix that is merged on master,
http://review.gluster.org/#/c/11316/5

Change-Id: I21e3fcffe8274986be53766e12a2c049eaa2da09
BUG: 1234216
Signed-off-by: Meghana M 
Reviewed-on: http://review.gluster.org/11337
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: soumya k 
Reviewed-by: Kaleb KEITHLEY