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

glusterd : allocate peerid to store in frame->cookie

2015-06-01T09:23:26+00:00

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

commit a1de3b05 was using peerid from the stack and storing it in the
frame->cookie and in the subsequent callback it was referred. The existance of
this variable is not guranteed in the cbk since its not dynamically allocated.
Fix is to dynmacially manage peerid in the frame cookie.

This patch also fixes one problem in gd_sync_task_begin () where unlock is not
triggered if the cluster is running with lesser than 3.6 op-version resulting into
commands failing with another transaction is in progress.

Change-Id: I0d22cf663df53ef3769585703944577461061312
BUG: 1223215
Signed-off-by: Atin Mukherjee 
Reviewed-on: http://review.gluster.org/10842
Tested-by: NetBSD Build System
Tested-by: Gluster Build System 
Reviewed-by: Kaushal M 
(cherry picked from commit 37f365843bed87728048da1f56de22290f5cb70f)
Reviewed-on: http://review.gluster.org/10959
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Krishnan Parthasarathi

quota: quota.conf backward compatibility fix

2015-06-01T09:13:35+00:00

This is a backport of http://review.gluster.org/#/c/10889/

> In release-3.7 the format of quota.conf is changed.
> There is a backward compatibility issues during upgrade
> 1) There can be an issue when peer sync between node-3.6 and node-3.7
> 2) If the user sets/removes limit, there is will different format of
>    file in node-3.6 and node-3.7
>
> This patch fixes the issue:
> 1) restrict the user to execute command quota enable, limit-usage, remove
> 2) write quota.conf in older format if op-version is less than 3.6
>
> Change-Id: Ib76f5a0a85394642159607a105cacda743e7d26b
> BUG: 1223739
> Signed-off-by: vmallika 
> Reviewed-on: http://review.gluster.org/10889
> Tested-by: NetBSD Build System
> Reviewed-by: Krishnan Parthasarathi 

Change-Id: I3d5075b96de1e125d30b0f6045389573aaaaecdb
BUG: 1226153
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/10985
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Krishnan Parthasarathi

glusterd/snapshot: Return correct errno in events of failure - PATCH 1

2015-06-01T07:18:15+00:00

       Backport of patch http://review.gluster.org/#/c/10313/

RETCODE              ERROR
-------------------------------------------
30800                Internal Error
30801                Another Transaction In Progress

>Change-Id: Ica7fd2e513b2c28717b6df73cfb2667725dbf057
>BUG: 1226117
>Signed-off-by: Avra Sengupta 
>Reviewed-on: http://review.gluster.org/10313
>Reviewed-by: Rajesh Joseph 
>Tested-by: Gluster Build System 
>Tested-by: NetBSD Build System
>Reviewed-by: Krishnan Parthasarathi 
>(cherry picked from commit 4397d7e72fdff6f01c59b72eebea421f23c1a392)

Change-Id: Iace6ac0f150919cead94e6c5d99a23d28d45046e
BUG: 1226117
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/11011
Tested-by: NetBSD Build System 
Reviewed-by: Kaushal M

glusterd: function to create duplicate of volinfo should copy subvol_count

2015-06-01T04:42:17+00:00

        Back port of http://review.gluster.org/10761

when we create duplicate volfile from a existing volfile,
we are not copying the variable subvol_count to the new
volfile.

 >Change-Id: I943aa7fdf1a2ca5bf57522cb2402b6b3165501ac
 >BUG: 1215002
 >Signed-off-by: Mohammed Rafi KC 
 >Reviewed-on: http://review.gluster.org/10761
 >Reviewed-by: Atin Mukherjee 
 >Tested-by: Gluster Build System 
 >Tested-by: NetBSD Build System

Change-Id: I3c58018833ad84fba13e1d17755f5dadbb01a5d3
BUG: 1226032
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/10982
Tested-by: NetBSD Build System 
Reviewed-by: Joseph Fernandes
Reviewed-by: Kaushal M

tiering/nfs: duplication of nodes in client graph

2015-06-01T04:40:47+00:00

        Back port of http://review.gluster.org/10820

When creating client volfiles, xlator tier-dht will
be loaded for each volume. So for services like nfs
have one or more volumes . So for each volume in the
graph a tier-dht xlator will be created. So the graph
parser will fail because of the redundant node in
graph.

By this change tier-dht will be renamed as volname-tier-dht

 >Change-Id: I3c9b9c23ddcb853773a8a02be7fd8a5d09a7f972
 >BUG: 1222840
 >Signed-off-by: Mohammed Rafi KC 
 >Reviewed-on: http://review.gluster.org/10820
 >Reviewed-by: Atin Mukherjee 
 >Tested-by: Gluster Build System 
 >Tested-by: NetBSD Build System
 >Reviewed-by: Kaushal M 

Change-Id: I5629d48d4d1dbec8790f75e2fee66729aa2f6eed
BUG: 1226029
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/10981
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Joseph Fernandes
Reviewed-by: Kaushal M

tiering/rebalance: Use separate pid/socket file for tiering

2015-06-01T04:38:26+00:00

        Back port of http://review.gluster.org/10792

When promotion/demotion daemon starts, it uses the same pidfile
as rebalance. This patch will introduce a different pid file
for the same.

 >Change-Id: Ic484c53f51e00ae6b2d697748a9600b14829e23b
 >BUG: 1221970
 >Signed-off-by: Mohammed Rafi KC 
 >Reviewed-on: http://review.gluster.org/10792
 >Reviewed-by: Atin Mukherjee 
 >Tested-by: Gluster Build System 
 >Tested-by: NetBSD Build System

Change-Id: Idda13e983ffd443672aee0873ee51e8cc7089c49
BUG: 1221969
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/10980
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Joseph Fernandes
Reviewed-by: Kaushal M

tier: Do not allow detach-tier commands on a non-tiered volume

2015-06-01T04:32:21+00:00

        Back port of http://review.gluster.org/10773

 >Change-Id: Ic92d25db68e40ef4a4388ef42affd1b3ee5a7ec6
 >BUG: 1221270
 >Signed-off-by: Mohammed Rafi KC 
 >Reviewed-on: http://review.gluster.org/10773
 >Reviewed-by: Atin Mukherjee 
 >Reviewed-by: Raghavendra G 
 >Reviewed-by: Kaushal M 
 >Tested-by: Gluster Build System 
 >Tested-by: NetBSD Build System
 >Signed-off-by: Mohammed Rafi KC 

Change-Id: I4b52da590dfcca8edc7e2b7e0c24c5dab7983c10
BUG: 1221967
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/10979
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Joseph Fernandes
Reviewed-by: Kaushal M

tiering: Correct errors in cli and glusterd

2015-06-01T04:31:53+00:00

        Back port of http://review.gluster.org/10768

Problem 1:

volume info shows Cold Bricks instead of Tier type
  eg:
Volume Name: patchy2
Type: Tier
Volume ID: 28c25b8d-b8a1-45dc-b4b7-cbd0b344f58f
Status: Started
Number of Bricks: 3
Transport-type: tcp
Hot Tier :
Hot Tier Type : Distribute
Number of Bricks: 1
Brick1: 10.70.1.35:/home/brick43
Cold Bricks:
Cold Tier Type : Distribute
Number of Bricks: 2
Brick2: 10.70.1.35:/home/brick19
Brick3: 10.70.1.35:/home/brick16
Options Reconfigured:

Problem 2: Detach-tier sending enums of Rebalance

   detach-tier has it's own Enum to send with detach-tier command,
using that enums will make more appropriate.

Problem 3:

Wrongly sets hot_brick count during the dictionary copying for response

>Change-Id: Icc054a999a679456881bc70511470d32ff8a86e4
>BUG: 1211264
>Signed-off-by: Mohammed Rafi KC 
>Reviewed-on: http://review.gluster.org/10768
>Reviewed-by: Atin Mukherjee 
>Reviewed-by: Kaushal M 
>Tested-by: Gluster Build System 
>Tested-by: NetBSD Build System

Change-Id: I889d3b83a1f197b79eb71e594a1ad1f2037ed402
BUG: 1226024
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/10978
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Joseph Fernandes
Reviewed-by: Kaushal M

features/bitrot: reimplement scrubbing frequency

2015-05-31T04:14:10+00:00

This patch reimplments existing scrub-frequency mechanism used
to schedule scrubber runs. Existing mechanism uses periodic
sleeps (waking up periodically on minimum granularity) and
performing a number of tracking checks based on counters and
sleep times. This patch does away with all the nifty counters
and uses timer-wheel to schedule scrub runs.

Scheduling changes are peformed by merely calculating the new
expiry time and calling mod_timer() [mod_timer_pending() in
some cases] making the code more debuggable and easier to
follow. This also introduces "hourly" scrubbing tunable as an
aid for testing scrubbing during development/testing cycle.

One could also implement on-demand scrubbing with ease: by
invoking mod_timer() with an expiry of one (1) second, thereby
scheduling a scrub run the very next second.

Change-Id: I6c7c5f0c6c9f886bf574d88c04cde14b76e60a8b
BUG: 1224647
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.org/10902
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System

glusterd: fix double-free of rebalance process' rpc object

2015-05-30T14:05:06+00:00

Change-Id: I0c79c4de47a160b1ecf3a8994eedc02e3f5002a9
BUG: 1225318
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/10872
Reviewed-by: Atin Mukherjee 
Reviewed-by: Kaushal M 
Reviewed-on: http://review.gluster.org/10932
Tested-by: NetBSD Build System
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur