glusterfs-afrv1.git/xlators/mgmt/glusterd/src/glusterd-volume-ops.c, branch release-3.3

glusterd: Fixed glusterd_brick_create_path algo.

2012-05-19T10:27:20+00:00

- check if any prefix of the brick path has "trusted.gfid"
  or "trusted.glusterfs.volume-id" set.
- set trusted.glusterfs.volume-id on the bricks as soon as
  its induction into the volume is settled. Earlier, the setting of
  "volume-id" used to happen during the first run of the brick process,
  leaving of window for bricks part of one volume to be (ab)used by another
  volume inadvertently.
- removed creation of brick directory (if missing), during start volume force.
  This is to avoid directory creation as part 'force'ful starting of volume
  and leave the responsibility with the user, who understands the
  'availability' of the export directory (brick) better.

Change-Id: I4237ec4ea7a4e38a7501027e7de7112edd67de8c
BUG: 812214
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.com/3280
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur 
Reviewed-on: http://review.gluster.com/3313

mgmt/glusterd: take the statedumps of running bricks if one of the bricks is down

2012-05-01T17:25:39+00:00

Till now if one of the bricks of the volume is down, then statedump of that
volume (bricks) did not take the statedumps of running bricks.

This patch fixes that issue and cli command will take the statedumps of
running brick processes.

Change-Id: Id9b11905d3d759202e505c2ecfa6ccd5dce6a9ff
BUG: 816884
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.com/3237
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

glusterd: Added volume-id to 'op' dictionary

2012-04-13T06:07:08+00:00

Volume-id passed in op dictionary would help detect possible split brains
among peers in a cluster. The idea is to check if the volume's id and
the vol-id that was passed are equal.
ie, same volume name, but different volume id indicate that glusterd
'metadata' of one of the participating peers is stale or there is
a split brain.

This is over and above the existing checksum based validation of peer
supplied cluster 'metadata' (ie, volume info file).

Change-Id: I1049ef249e417e540ccb4243e450f92fcd0f46f9
BUG: 797734
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.com/3083
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

Fix compiler warnings and typos from Debian build.

2012-04-10T21:37:29+00:00

Mostly to do with "-Werror=format-security" being buggy, but while we're
here we might as well fix some typos and such.  Credit goes to Patrick
Matthäi  for pointing these out.

Change-Id: Ia32d1111d7c10b1f213df85d86b17a1326248ffd
BUG: 811387
Signed-off-by: Jeff Darcy 
Reviewed-on: http://review.gluster.com/3117
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

mgmt/glusterd: Improvements for volume heal CLI.

2012-04-01T12:53:42+00:00

1) Brought in a CLI improvement for volume heal cmd when it fails
before initiating a transaction.

2) Minor improvements in UI messages related to self-heal-daemon.

Change-Id: I983c1e54b48f2efe8793fa5b00fa50df2067683b
BUG: 807261
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.com/3057
Tested-by: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri

glusterd: bring back the 'non-synctask' behavior

2012-03-22T11:30:36+00:00

revert back to non synctask behavior

Change-Id: Icfc9cbdeaf8dd6735df8366a6121c6598c56fade
Signed-off-by: Amar Tumballi 
BUG: 805802
Reviewed-on: http://review.gluster.com/3002
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

Add space for heal vol req log message.

2012-03-22T05:38:36+00:00

Change-Id: Iadf1d6304bcae8f666f2435ffa2d71c50dd1395a
BUG: 764658
Signed-off-by: Rahul C S 
Reviewed-on: http://review.gluster.com/2991
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Raghavendra Bhat

glusterd: Disabled self-heal on clear-locks internal mount

2012-03-21T18:59:57+00:00

- Also, changed afr_get_xattr_clrlk_cbk to use dict_set_dynstr for
  clear-lock summary. Earlier, it was relying on 'str' passed from
  xlators below.

Change-Id: I175f4542e6ef2c859c4811eecb9d8c5a7d25a283
BUG: 800779
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.com/2992
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Anand Avati

glusterd: bring in feature to use syncop for mgmt ops

2012-03-21T18:58:11+00:00

* new sycnop routines added to mgmt program
* one should not use 'glusterd_op_begin()', instead can use the
  synctask framework, 'glusterd_op_begin_synctask()'
* currently using for below operations:
  'volume start', 'volume rebalance', 'volume quota',
  'volume replace-brick' and 'volume add-brick'

Change-Id: I0bee76d06790d5c5bb5db15d443b44af0e21f1c0
BUG: 762935
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.com/479
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

mgmt/glusterd: don't create the brick path in 'volume start'

2012-03-18T08:09:25+00:00

create the missing brick directory only on a
'gluster volume start  force' command

Change-Id: Ica4fab9cefe953d73de9a30843d48f94e6ab735c
Signed-off-by: Amar Tumballi 
BUG: 801610
Reviewed-on: http://review.gluster.com/2921
Tested-by: Gluster Build System 
Reviewed-by: Rajesh Amaravathi 
Reviewed-by: Anand Avati