glusterfs.git/tests, branch v3.4.0qa8

cluster/dht: stack wind with cookie

2013-02-01T01:18:03+00:00

Default_fops uses stack_wind_tail. It winds without creating the frame leading
into wrong subvol return in the cookie. To avoid the problem caused by the
same, we're getting the subvol by passing the cookie.

Change-Id: I51ee79b22c89e4fb0b89e9a0bc3ac96c5b469f8f
BUG: 893338
Signed-off-by: Varun Shastry 
Reviewed-on: http://review.gluster.org/4388
Reviewed-by: Jeff Darcy 
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati 
Tested-by: Anand Avati

fuse-bridge: fix some breakages from lock migration patch

2013-01-31T07:43:49+00:00

- do not attempt lock migration if no locks were ever acquired on
  an fd.

- fix fd_lk_ctx_t ref leak during fd migration

- remove spurious fd_unref() (probably added to compensate for
  the fd_ref leak in syncop_open_cbk)

- remove @newfdptr out-param which makes fd ref management really
  tricky (and currently refs were unmanaged for the out-param).
  Instead acquire ref and unref within lock migration function.

Change-Id: I4cc9c451f0df4c051612bd1fa7bef11e801570e4
BUG: 808400
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/4453
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

762989.t: fix a typo by grepping only the blocked port number from netstat o/p

2013-01-30T18:40:41+00:00

Change-Id: I14c3daa799f0936014bf8aa9fd798ecbaec6e912
BUG: 762989
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/4426
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

glusterfs : Moved option files, and statedumps from /tmp

2013-01-29T23:27:29+00:00

Change-Id: Ibdede396c4d6859225937316b7a59a661bcaf9f5
BUG: 764890
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/4422
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/afr: if a subvolume is down wind the lock request to next

2013-01-29T20:50:55+00:00

When one of the subvolume is down, then lock request is not attempted
on that subvolume and move on to the next subvolume.

/* skip over children that are down */
                while ((child_index < priv->child_count)
                       && !local->child_up[child_index])
                        child_index++;

In the above case if there are 2 subvolumes and 2nd subvolume is down (subvolume
1 from afr's view), then after attempting lock on 1st child (i.e subvolume 0)
child index is calculated to be 1. But since the 2nd child is down child_index
is incremented to 2 as per the above logic and lock request is STACK_WINDed to
the child with child_index 2. Since there are only 2 children for afr the child
(i.e the xlator_t pointer) for child_index will be NULL. The process crashes
when it dereference the NULL xlator object.

Change-Id: Icd9b5ad28bac1b805e6e80d53c12d296526bedf5
BUG: 765564
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/4438
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

Tests: Test functionality of flush/fsync with afr

2013-01-29T20:34:16+00:00

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

quick-read: various fixes

2013-01-29T19:56:42+00:00

- initialize xdata in qr_lookup even if it was NULL from top. This
  allows qr to do its job even if lookup originated from fuse-resolve.c

- extend test cases to include 1 second delay and retry

- fix bug while checking condition for cached unwind

  qr_readv_cached() unwinds if op_ret > 0. Therefore qr_readv()
  must wind to subvol only if !(op_ret > 0) (i.e, op_ret <= 0).

- qr_readv_cached() is using uninitialized @conf pointer. Thanks
  to Raghavendra Bhat for catching this!

Change-Id: Ifaf2ea2685e452210ef9ba3c2d1f2ab51900650c
BUG: 846240
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/4452
Reviewed-by: Raghavendra Bhat 
Tested-by: Gluster Build System

performance/io-cache: propagate errors while unwinding frame in

2013-01-29T19:01:34+00:00

                      read path.

Change-Id: Ieb5d592a987e8681d5ec019da309f75e3b207580
BUG: 858242
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/4204
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

tests: skip time consuming mock builds for code-only changes

2013-01-29T18:29:03+00:00

tests/basic/rpm.t takes a bulk of the time for the regression tests.
Often the building of the rpm is not affected at all and therefor the
tests does not add any value.

With this change the rpmbuild/mock test will only be run when some
changes affect the build system. Changes affecting 'tests/', 'doc/',
'*.py', *.c' and '*.h' do not trigger the test anymore.

Change-Id: Ic188b9e26cde3113b2bdf9cd1fab56d9fd85a4b7
BUG: 904005
Signed-off-by: Niels de Vos 
Reviewed-on: http://review.gluster.org/4429
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

mgmt/glusterd: Expose post-op-delay through cli

2013-01-29T07:25:34+00:00

Change-Id: I13e3699bd58d53896ae54e1bfafb3cd1c9580c7c
BUG: 905307
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/4443
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Gluster Build System