glusterfs.git, branch v3.4.0qa5

glusterd: Set op-version for quorum options

2012-12-12T01:17:20+00:00

Change-Id: I693cc96c1dc7dc560c3c25698f08b846e8a48fca
BUG: 839595
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.org/4291
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

Added virt group file - template for 'group' set

2012-12-12T00:42:14+00:00

- This sample file contains the set of volume options
  that are associated with volumes that would be used to
  store VMs.

- To enable the set of options present in this file one must
  do the following,
  - # cp extras/group-virt.example /var/lib/glusterd/groups/
    # gluster volume set  group 

Change-Id: I8848d3d222ce114900db885aeef2f20c83ece28c
BUG: 851237
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/4232
Reviewed-by: Anand Avati 
Tested-by: Gluster Build System

gsyncd / geo-rep: do not start geo-rep if replace brick is in progress

2012-12-12T00:26:52+00:00

Change-Id: I9db32544ceb6f90c8231aaf40d722f6869a72614
BUG: 861945
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.org/4289
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/afr: Remember type of split-brain in inode-ctx

2012-12-12T00:23:53+00:00

Along with this change, fixed the race of setting the
split-brain status in inode-ctx after unwinding the fop from
self-heal in case of back-ground self-heal.

Change-Id: Ifc829300df485f50f139443802e8b6dc7038b4ad
BUG: 873962
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/4198
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

glusterd: log appropriate message when locking fails

2012-12-12T00:06:08+00:00

PROBLEM:

When a transaction is already in progress, and the user tries to
execute another glusterd operation, the second operation fails as
glusterd fails to acquire lock. But to the user, a message like
"Operation failed" does not give ample information about why the
operation failed.

FIX:

Made glusterd_op_txn_begin use and initialise error string, which is
needed to capture failure in the "lock" phase.

Also made gd_sync_task_begin set error string appropriately when
locking fails.

In the process, I had to introduce error string in some glusterd_handle_*
functions. And because I introduced error string in these handlers, I
decided to also set them in places where these handlers could possibly
fail.

HOW I TESTED IT:

For want of a better idea, I "commented out" the call to
"glusterd_unlock", recompiled glusterd and ran two glusterd volume
operations, one after the other. The second operation fails with the
message "Another transaction is in progress. Please try again after
sometime." as expected.

The tests were performed on two volume ops : one of them
synctask'ized (volume start) and the other NOT (volume create).

Change-Id: Ia862972929872ae2f053707a544824d9cadc37be
BUG: 873549
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/4197
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/distribute: re-set layouts to prevent overlaps

2012-12-12T00:05:50+00:00

When subvols-per-directory option is used, with bricks addition/removal
the layouts might get distributed to other subvols, which were not part
of the layout before. We need to clean up layouts on old subvolumes, to
prevent overlaps.

Also, we need to make sure if layout-cnt is never less
than subvolume-cnt.

Change-Id: I00994a092ca0c99aedcc41bd9412d43460f88a04
BUG: 884455
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/4281
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

Synchronize ls-loop with main-script

2012-12-12T00:05:27+00:00

Change-Id: If9a15e4c272baa28eee4f2fdfdf52b2e032b3a96
BUG: 884452
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/4292
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

object-storage: unit tests to 100% cover utils

2012-12-11T12:53:53+00:00

Change-Id: I2d540e16cc3d420b9bb88e6857e9795d56e34529
BUG: 884327
Signed-off-by: Peter Portante 
Reviewed-on: http://review.gluster.org/4225
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Mohammed Junaid 
Tested-by: Gluster Build System

object-storage: use temp file optimization for pkl

2012-12-11T12:53:27+00:00

Override OpenStack Swift's swift.common.utils.write_pickle with our own
implementation that uses the GlusterFS temporary file operation. A file name
'..' will hash to the same GlusterFS node as a file named
'', those avoiding a move of the file on a rename.

This is part of the work needed to address BZ 876660
(https://bugzilla.redhat.com/show_bug.cgi?id=876660).

Change-Id: I1cb9f97f289ab2ca76ec9221366df74de08268bb
BUG: 876660
Signed-off-by: Peter Portante 
Reviewed-on: http://review.gluster.org/4224
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Mohammed Junaid 
Tested-by: Gluster Build System

cli: fail vol set when value = empty string/string with all whitespaces

2012-12-10T22:18:34+00:00

PROBLEMS:

a. 'volume set' operation accepts empty strings and strings containing
   all whitespaces for value. The result - subsequent attempts to start
   glusterd fail.

b. 'volume set group' operation, when passed an empty string or string
   containing all spaces as value, fopens the wrong file.

FIX:

Made volume set parser in cli check for and invalidate value if its length
is 0 after all the whitespaces have been stripped off.

Change-Id: I6792ee1fe9452d8e25e1fd1eeb1b9981be3389f1
BUG: 859927
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/4246
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati