|
 |
233933 |
From 7a04fb9999f5d25c17f5593eed5e98d0f5a1932d Mon Sep 17 00:00:00 2001
|
|
|
233933 |
From: karthik-us <ksubrahm@redhat.com>
|
|
|
233933 |
Date: Mon, 15 Jul 2019 14:30:52 +0530
|
|
|
233933 |
Subject: [PATCH 251/255] tests: Fix
|
|
|
233933 |
bug-1717819-metadata-split-brain-detection.t failure
|
|
|
233933 |
|
|
|
233933 |
<Backport of: https://review.gluster.org/#/c/glusterfs/+/23043/>
|
|
|
233933 |
|
|
|
233933 |
Problem:
|
|
|
233933 |
tests/bugs/replicate/bug-1717819-metadata-split-brain-detection.t fails
|
|
|
233933 |
intermittently in test cases #49 & #50, which compare the values of the
|
|
|
233933 |
user set xattr values after enabling the heal. We are not waiting for
|
|
|
233933 |
the heal to complete before comparing those values, which might lead
|
|
|
233933 |
those tests to fail.
|
|
|
233933 |
|
|
|
233933 |
Fix:
|
|
|
233933 |
Wait till the HEAL-TIMEOUT before comparing the xattr values.
|
|
|
233933 |
Also cheking for the shd to come up and the bricks to connect to the shd
|
|
|
233933 |
process in another case.
|
|
|
233933 |
|
|
|
233933 |
Change-Id: I0021c2d5d251111c695e2bf18c63e8189e456114
|
|
|
233933 |
fixes: bz#1704562
|
|
|
233933 |
Signed-off-by: karthik-us <ksubrahm@redhat.com>
|
|
|
233933 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/176071
|
|
|
233933 |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
233933 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
233933 |
---
|
|
|
233933 |
tests/bugs/replicate/bug-1717819-metadata-split-brain-detection.t | 6 ++++++
|
|
|
233933 |
1 file changed, 6 insertions(+)
|
|
|
233933 |
|
|
|
233933 |
diff --git a/tests/bugs/replicate/bug-1717819-metadata-split-brain-detection.t b/tests/bugs/replicate/bug-1717819-metadata-split-brain-detection.t
|
|
|
233933 |
index 94b8bf3..76d1f21 100644
|
|
|
233933 |
--- a/tests/bugs/replicate/bug-1717819-metadata-split-brain-detection.t
|
|
|
233933 |
+++ b/tests/bugs/replicate/bug-1717819-metadata-split-brain-detection.t
|
|
|
233933 |
@@ -76,6 +76,10 @@ EXPECT_WITHIN $HEAL_TIMEOUT "^1$" get_pending_heal_count $V0
|
|
|
233933 |
|
|
|
233933 |
# Launch heal
|
|
|
233933 |
TEST $CLI volume heal $V0 enable
|
|
|
233933 |
+EXPECT_WITHIN $PROCESS_UP_TIMEOUT "^Y$" glustershd_up_status
|
|
|
233933 |
+EXPECT_WITHIN $CHILD_UP_TIMEOUT "^1$" afr_child_up_status_in_shd $V0 0
|
|
|
233933 |
+EXPECT_WITHIN $CHILD_UP_TIMEOUT "^1$" afr_child_up_status_in_shd $V0 1
|
|
|
233933 |
+EXPECT_WITHIN $CHILD_UP_TIMEOUT "^1$" afr_child_up_status_in_shd $V0 2
|
|
|
233933 |
TEST $CLI volume heal $V0
|
|
|
233933 |
EXPECT_WITHIN $HEAL_TIMEOUT "^0$" get_pending_heal_count $V0
|
|
|
233933 |
|
|
|
233933 |
@@ -117,6 +121,8 @@ EXPECT_WITHIN $PROCESS_UP_TIMEOUT "^Y$" glustershd_up_status
|
|
|
233933 |
EXPECT_WITHIN $CHILD_UP_TIMEOUT "^1$" afr_child_up_status_in_shd $V0 0
|
|
|
233933 |
EXPECT_WITHIN $CHILD_UP_TIMEOUT "^1$" afr_child_up_status_in_shd $V0 1
|
|
|
233933 |
EXPECT_WITHIN $CHILD_UP_TIMEOUT "^1$" afr_child_up_status_in_shd $V0 2
|
|
|
233933 |
+TEST $CLI volume heal $V0
|
|
|
233933 |
+EXPECT_WITHIN $HEAL_TIMEOUT "^0$" get_pending_heal_count $V0
|
|
|
233933 |
|
|
|
233933 |
B0_XATTR=$(getfattr -n 'user.metadata' --absolute-names --only-values $B0/${V0}0/dir/file)
|
|
|
233933 |
B1_XATTR=$(getfattr -n 'user.metadata' --absolute-names --only-values $B0/${V0}1/dir/file)
|
|
|
233933 |
--
|
|
|
233933 |
1.8.3.1
|
|
|
233933 |
|