glusterfs.git/xlators/mgmt, branch v3.7.2

glusterd: Fix snapshot of a volume with geo-rep

2015-06-19T06:37:31+00:00

Problem:
    Snapshot fails for a volume configured with geo-rep
    if geo-rep is created with root user with the following
    syntax.
    gluster vol geo-rep  root@::

    It works fine if created with following syntax.
    gluster vol geo-rep  ::

Cause:
    Geo-rep maintains persistent dictionary of slave associated
    with master volume. The dictionary saves the slave info
    along with 'root@' as sent from cli.
    Snapshot while constructing the working dir path to copy
    configuration files, constructs using this dictionary.
    But the actual working dir is created with out considering
    'root@'. Hence the issue.

Fix:
    Fix is done at two layers.
    1. Parse and negelect 'root@' in cli itself.
    2. For existing geo-rep sessions and upgrade scenarios,
       parse and neglect 'root@' in snapshot code as well.

BUG: 1233056
Change-Id: Ic666e43a1c4ac2f3f0fb957caa77770e1eee4129
Signed-off-by: Kotresh HR 
Reviewed-on: http://review.gluster.org/11233
Reviewed-on: http://review.gluster.org/11310
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi

glusterd: Buffer overflow causing crash for glusterd

2015-06-19T01:30:28+00:00

Backport of http://review.gluster.org/11090

Problem: In GLUSTERD_GET_DEFRAG_PROCESS we are using PATH_MAX (4096)
as the max size of the input for target path, but we have allocated
NAME_MAX (255) size of buffer for the target.

Now this crash is not seen with source, but seen with RPMS.
The reason is _foritfy_fail. This check happens when _FORTIFY_SOURCE
is enabled. This option tries to figure out possible
overflow scenarios like the bug here and does crash the process.

BUG: 1227677
Change-Id: I50cf83cb60c640e46cc7a1a8d3a8321b9147fba9
Signed-off-by: Susant Palai 
Reviewed-on: http://review.gluster.org/11091
Reviewed-by: Atin Mukherjee 
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System

NFS-Ganesha: Automatically export vol that was exported before vol restart

2015-06-18T17:24:42+00:00

Consider a volume that is exported via NFS-Ganesha. Stopping this
volume will automatically unexport the volume. Starting this volume
should automatically export it. Although the logic was already there,
there was a bug in it. Fixing the same by introducing a hook script.

Also with the new CLI options, the hook script S31ganesha-set.sh
is no longer required. Hence, removing the same.
Adding a comment to tell the user that one of the CLI
commands will take a few minutes to complete.

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

Change-Id: I1c16a978b9e2093a4298ea3024d031fd1a5bb577
BUG: 1232335
Signed-off-by: Meghana Madhusudhan 
Reviewed-on: http://review.gluster.org/11259
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: soumya k 
Reviewed-by: Niels de Vos

quota/glusterd: porting to new logging framework.

2015-06-18T10:05:51+00:00

Backport of http://review.gluster.org/#/c/10473/8

Change-Id: I879a181b6d6b97119a6b4fab90a12b8386635383
BUG: 1217722
Signed-off-by: Nandaja Varma 
Reviewed-on: http://review.gluster.org/11222
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Sachin Pandit 
Reviewed-by: Atin Mukherjee

bitrot/glusterd: gluster volume set command for bitrot should not supported

2015-06-18T06:53:02+00:00

Currently gluster volume set  bitrot succeeds. gluster volume
set command for bitrot is not supported.
Gluster should only accept gluster volume bitrot  * commands.

Change-Id: I5ff4b79f202ad018c76188f19d6311aad0d7c166
BUG: 1232589
Signed-off-by: Gaurav Kumar Garg 
Reviewed-on: http://review.gluster.org/11118
Tested-by: Gluster Build System 
Reviewed-by: Atin Mukherjee 
Tested-by: Atin Mukherjee 
(cherry picked from commit 1ac3d28d8dde6360550c80a10d8add572937be16)
Reviewed-on: http://review.gluster.org/11265

features/bitrot: tuanble object signing waiting time value for bitrot

2015-06-17T16:38:59+00:00

 Currently bitrot using 120 second waiting time for object to be signed
 after all fop's released. This signing waiting time value should be tunable.

 Command for changing the signing waiting time will be
 #gluster volume bitrot  signing-time 

Change-Id: I89f3121564c1bbd0825f60aae6147413a2fbd798
BUG: 1231832
Signed-off-by: Gaurav Kumar Garg 
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.org/11105
(cherry picked from commit 554fa0c1315d0b4b78ba35a2d332d7ac0fd07d48)
Reviewed-on: http://review.gluster.org/11235
Tested-by: Gluster Build System

tier/volume set: Validate volume set option for tier

2015-06-17T05:13:18+00:00

Volume set option related to tier volume can only be set
for tier volume, also currently all volume set i for tier
option accepts a non-negative integer. This patch validate
both condition.

Back port of:
>Change-Id: I3611af048ff4ab193544058cace8db205ea92336
>BUG: 1216960
>Signed-off-by: Mohammed Rafi KC 
>Signed-off-by: Dan Lambright 
>Reviewed-on: http://review.gluster.org/10751
>Tested-by: Gluster Build System 
>Tested-by: NetBSD Build System 
>Reviewed-by: Joseph Fernandes


(cherry picked from commit f6a062044a3447bea5bf0fcf21a3f85c00fb6c7d)

Change-Id: Ic6081f0ce7ae7effac69ba192bd35c8d382a11d5
BUG: 1230560
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/11173
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Joseph Fernandes
Tested-by: Joseph Fernandes

glusterd/tier: glusterd crashed with detach-tier commit force

2015-06-17T05:10:55+00:00

glusterd crashed when doing "detach-tier commit force" on a
non-tiered volume.

>Change-Id: I884771893bb80bec46ae8642c2cfd7e54ab116a6
>BUG: 1228112
>Signed-off-by: Mohammed Rafi KC 
>Reviewed-on: http://review.gluster.org/11081
>Tested-by: Gluster Build System 
>Tested-by: NetBSD Build System 
>Reviewed-by: Joseph Fernandes
>Reviewed-by: Dan Lambright 
>Reviewed-by: Krishnan Parthasarathi 

(cherry picked from commit fefc8730e2a87db41d5f79c9853a14cab86715e5)

Change-Id: Ia536da3ec4c632d7d1cc1575dd2b7b098a2c7aec
BUG: 1230563
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/11174
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Joseph Fernandes
Tested-by: Joseph Fernandes

rebalance,store,glusterd/glusterd: porting to new logging framework.

2015-06-17T04:23:24+00:00

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

Change-Id: I231b79e3414e60fe67cde577dd585cce83c8bfad
BUG: 1217722
Signed-off-by: Nandaja Varma 
Reviewed-on: http://review.gluster.org/11193
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Atin Mukherjee

sm/glusterd: Porting messages to new logging framework

2015-06-17T04:14:09+00:00

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

(cherry picked from commit 72a7a6ea78289b2897f9846dc4e111f442dd2788)
>Change-Id: I391d1ac6a7b312461187c2e8c6f14d09a0238950
>BUG: 1194640
>Signed-off-by: Nandaja Varma 
>Reviewed-on: http://review.gluster.org/9927
>Reviewed-by: Atin Mukherjee 
>Tested-by: NetBSD Build System 
>Tested-by: Gluster Build System 
>Reviewed-by: Krishnan Parthasarathi 

Change-Id: I391d1ac6a7b312461187c2e8c6f14d09a0238950
BUG: 1217722
Signed-off-by: Nandaja Varma 
Reviewed-on: http://review.gluster.org/11197
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Atin Mukherjee