glusterfs.git/xlators/mgmt/glusterd/src, branch v3.7.4

glusterd: Don't allow remove brick start/commit if glusterd is down of the host of the brick

2015-08-28T09:02:29+00:00

Backport of http://review.gluster.org/#/c/11726/

remove brick stage blindly starts the remove brick operation even if the
glusterd instance of the node hosting the brick is down. Operationally its
incorrect and this could result into a inconsistent rebalance status across all
the nodes as the originator of this command will always have the rebalance
status to 'DEFRAG_NOT_STARTED', however when the glusterd instance on the other
nodes comes up, will trigger rebalance and make the status to completed once the
rebalance is finished.

This patch fixes two things:
1. Add a validation in remove brick to check whether all the peers hosting the
bricks to be removed are up.

2. Don't copy volinfo->rebal.dict from stale volinfo during restore as this
might end up in a incosistent node_state.info file resulting into volume status
command failure.

Change-Id: Ia4a76865c05037d49eec5e3bbfaf68c1567f1f81
BUG: 1256265
Signed-off-by: Atin Mukherjee 
Reviewed-on: http://review.gluster.org/11726
Tested-by: NetBSD Build System 
Reviewed-by: N Balachandran 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/11996
Tested-by: Gluster Build System 
Reviewed-by: Kaushal M

glusterd: stop all the daemons services on peer detach

2015-08-26T11:58:03+00:00

Backport of:  http://review.gluster.org/#/c/11509/

Currently glusterd is not stopping all the deamon service on peer detach

With this fix it will do peer detach cleanup properlly and will stop all
the daemon which was running before peer detach on the node.

Change-Id: Ifed403ed09187e84f2a60bf63135156ad1f15775
BUG: 1238706
Signed-off-by: Gaurav Kumar Garg 
Reviewed-on: http://review.gluster.org/11971
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Avra Sengupta 
Reviewed-by: Krishnan Parthasarathi

glusterd : Display status of Self Heal Daemon for disperse volume

2015-08-25T17:59:48+00:00

 Problem : Status of Self Heal Daemon is not
 displayed in "gluster volume status"

 Fix : As disperse volumes are self heal compatible,
 show the status of self heal daemon in gluster
 volume status command

Change-Id: I83d3e6a2fd122b171f15cfd76ce8e6b6e00f92e2
BUG: 1228216
Signed-off-by: Ashish Pandey 
Reviewed-on: http://review.gluster.org/10994
Tested-by: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri

snapshot: Log deletion of snapshot, during auto-delete

2015-08-24T06:51:05+00:00

    Backport of http://review.gluster.org/#/c/11963/

When auto-delete is enabled, and soft-limit is reached,
on creation of a snapshot, the oldest snapshot for that
volume is deleted.

Displaying a warning log before deleting the oldest
snapshot.

Change-Id: I75f0366935966a223b63a4ec5ac13f9fe36c0e82
BUG: 1255311
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/11964
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: mohammed rafi  kc 
Reviewed-by: Rajesh Joseph

tiering/glusterd: start tier daemon during volume start

2015-08-19T08:47:35+00:00

Tier daemon should always run with tier volume. If volume
is stopped and started again, we manually need to start
the tier-daemon, instead this patch will automatically trigger
tier process along with volume start.

A snapshot restored volume will not have node_state_info,
so we need to create and store it dynamically

Back port of:
>Change-Id: I659387c914bec7a1b6929ee5cb61f7b406402075
>BUG: 1238593
>Signed-off-by: Mohammed Rafi KC 
>Signed-off-by: Dan Lambright 
>Reviewed-on: http://review.gluster.org/11525
>Tested-by: NetBSD Build System 
>Tested-by: Gluster Build System 
>Reviewed-by: Krishnan Parthasarathi 

Change-Id: I5766a0712a5695423b42a28a0f2e7435f4dd2628
BUG: 1254442
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/11952
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Dan Lambright 
Tested-by: Dan Lambright

glusterd/tier: configure tier daemon during volume restart

2015-08-19T08:11:03+00:00

rebalance daemon will be running on every tier volume for
promoting/demoting the files. When volume/glusterd is restarted,
then we need to configure the daemon.

Back port of >
>Change-Id: Ib565240a70edea2ec8bc1601c52b40c0783491d3
>BUG: 1225330
>Signed-off-by: Mohammed Rafi KC 
>Signed-off-by: Joseph Fernandes 
>Reviewed-on: http://review.gluster.org/10933
>Reviewed-by: Dan Lambright 
>Tested-by: Dan Lambright 

(cherry picked from commit 7aaa414c67be460aca3c1c5eed047e3db17bd435)

Change-Id: Id15f5a18fd2ebaf8642b02561ae3666e619e3f83
BUG: 1235202
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/11376
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Dan Lambright 
Tested-by: Dan Lambright

Set nfs.disable to "on" when global NFS-Ganesha key is enabled

2015-08-18T12:12:33+00:00

"nfs.disable" gets set to "on" for all the existing volumes,
when the command "gluster nfs-ganesha enable" is executed.
When a new volume is created,it gets exported via Gluster-NFS on
the nodes outside the NFS-Ganesha. To fix this,
the "nfs.disable" key is set to "on" before starting the volume,
whenever the global option is set to "enable".

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

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

glusterd: Stop/restart/notify to daemons(svcs) during reset/set on a volume

2015-08-18T10:58:18+00:00

problem : Reset/set commands were not working properly. reset command returns
success but it not sending notification to svcs if corresponding graph modified.

Fix: Whenever reset/set command issued, generate the temp graph and compare
with original graph and do the fallowing actions
1.) If both graph are identical nothing to do with svcs.
2.) If any changes in graph topology restart/stop service by calling
svc manager.
3)  If changes in options send notify signal by calling glusterd_fetchspec_notify.

Back port of:
>Change-Id: I852c4602eafed1ae6e6a02424814fe3a83e3d4c7
>BUG: 1209329
>Signed-off-by: anand 
>Reviewed-on: http://review.gluster.org/10850
>Tested-by: NetBSD Build System 
>Tested-by: Gluster Build System 
>Reviewed-by: Atin Mukherjee 
>(cherry picked from commit 7255febab2c38cc89b71f2519a20d10f53586000)

Change-Id: I42aa757ecc6b5b307b5927d11f12d08f57ac0ae2
BUG: 1253165
Reviewed-on: http://review.gluster.org/11905
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Atin Mukherjee

glusterd: log improvement in glusterd_peer_rpc_notify

2015-08-17T07:23:24+00:00

Backport of http://review.gluster.org/#/c/10272/

If ping time out is enabled glusterd can receive a disconnect event from a peer
which has been already deleted resulting into a critical log printed. This patch
ensures that critical message is logged only when its a connect event.

Change-Id: I67d9aa3f60195e08af7dfc8a42683422aaf90a00
BUG: 1252680
Signed-off-by: Atin Mukherjee 
Reviewed-on: http://review.gluster.org/10272
Tested-by: Gluster Build System 
Reviewed-by: Gaurav Kumar Garg 
Tested-by: NetBSD Build System 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/11886
Reviewed-by: Anand Nekkunti

glusterd/rebalance: trusted rebalance volfile

2015-08-17T07:17:15+00:00

Creating the client volfiles with GF_CLIENT_OTHER
overwrites the trusted rebalance volfile and causes rebalance
to fail if auth.allow is set.
Now, we always set the value of trusted-client to GF_CLIENT_TRUSTED
for rebalance volfiles.

Change-Id: I95eb510256d18dfa9048f96a1aeb71cca4811811
BUG: 1253542
Signed-off-by: N Balachandran 
Reviewed-on: http://review.gluster.org/11819
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Avra Sengupta 
Reviewed-by: Rajesh Joseph 
Reviewed-by: Atin Mukherjee 
(cherry picked from commit 8603ac80218d6161416fd0f3cca511ba09aef2c8)
Signed-off-by: Nithya Balachandran 
Reviewed-on: http://review.gluster.org/11915