diff options
| author | Pranith Kumar K <pkarampu@redhat.com> | 2015-04-27 01:20:02 +0530 |
|---|---|---|
| committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2015-05-04 21:48:13 -0700 |
| commit | e6f2472d2434ab43a30720ef4de2e0abc0a3f4ac (patch) | |
| tree | a9ee3300d0720a3a710ced742f3b8d476fe5865b | |
| parent | c8cd488b794d7abb3d37f32a6d8d0a3b365aa46e (diff) | |
libglusterfs: Fix cluster_entrylk retry
Change-Id: I92ff46bae36d39a449d4bbaedc88a322992f65eb
BUG: 1215265
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/10391
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Krutika Dhananjay <kdhananj@redhat.com>
| -rw-r--r-- | libglusterfs/src/cluster-syncop.c | 6 | ||||
| -rwxr-xr-x | tests/bugs/bitrot/1207029-bitrot-daemon-should-start-on-valid-node.t | 6 |
2 files changed, 7 insertions, 5 deletions
diff --git a/libglusterfs/src/cluster-syncop.c b/libglusterfs/src/cluster-syncop.c index 9525bc55510..10993e6088d 100644 --- a/libglusterfs/src/cluster-syncop.c +++ b/libglusterfs/src/cluster-syncop.c @@ -1172,9 +1172,9 @@ cluster_entrylk (xlator_t **subvols, unsigned char *on, int numsubvols, cluster_unentrylk (subvols, locked_on, numsubvols, replies, output, frame, this, dom, inode, name); - FOP_ONLIST (subvols, on, numsubvols, replies, - locked_on, frame, entrylk, dom, &loc, name, - ENTRYLK_LOCK, ENTRYLK_WRLCK, NULL); + FOP_SEQ (subvols, on, numsubvols, replies, + locked_on, frame, entrylk, dom, &loc, name, + ENTRYLK_LOCK, ENTRYLK_WRLCK, NULL); break; } } diff --git a/tests/bugs/bitrot/1207029-bitrot-daemon-should-start-on-valid-node.t b/tests/bugs/bitrot/1207029-bitrot-daemon-should-start-on-valid-node.t index e5b53bd0d24..120facff02e 100755 --- a/tests/bugs/bitrot/1207029-bitrot-daemon-should-start-on-valid-node.t +++ b/tests/bugs/bitrot/1207029-bitrot-daemon-should-start-on-valid-node.t @@ -10,6 +10,9 @@ cleanup; +function get_bitd_count { + ps auxw | grep glusterfs | grep bitd.pid | grep -v grep | wc -l +} ## Start a 2 node virtual cluster TEST launch_cluster 2; @@ -30,8 +33,7 @@ TEST $CLI_1 volume bitrot $V0 enable ## Bitd daemon should be running on the node which is having brick. Here node1 ## only have brick so bitrot daemon count value should be 1. -bitrot_daemon=$(ps auxww | grep glusterfs | grep bitd.pid | grep -v grep | wc -l) -TEST [ "$bitrot_daemon" -eq 1 ]; +EXPECT_WITHIN $PROCESS_UP_TIMEOUT "1" get_bitd_count ## Bitd daemon should not run on 2nd node and it should not create bitrot ## volfile on this node. Below test case it to check whether its creating bitrot |