diff options
| author | Xavier Hernandez <xhernandez@datalab.es> | 2014-09-18 18:42:34 +0200 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2014-10-03 02:01:26 -0700 |
| commit | a97ad9b69bb17f2351c59512fa9c6cb25d82b4da (patch) | |
| tree | b13239ea132df6efb4185a5dfd0cfea18dc0b28e /tests/basic/ec/statedump.t | |
| parent | 7e76bd24305ee29506dd11be5a0691cfa65b6d0c (diff) | |
| download | glusterfs-a97ad9b69bb17f2351c59512fa9c6cb25d82b4da.tar.gz glusterfs-a97ad9b69bb17f2351c59512fa9c6cb25d82b4da.tar.xz glusterfs-a97ad9b69bb17f2351c59512fa9c6cb25d82b4da.zip | |
test/ec: Fix spurious failures caused by self-heal
The sha1sum of a file may update the access time of that file.
If this happens while a brick is down, as it is forced in the
test, that brick doesn't get the update, getting out of sync.
When the brick is restarted, self-heal repairs the file, but
the test shouldn't access brick contents until self-heal finishes.
If this is combined with a kill of another brick before self-heal
has finished repairing the file, the volume could become inaccessible.
Since the purpose of these tests is only to check ec functionality
(there is another test that checks self-heal), the test that corrupts
the file has been removed.
Additional checks to validate the state of the volume have been added
to avoid some timing issues.
BUG: 1144108
Change-Id: Ibd9288de519914663998a1fbc4321ec92ed6082c
Signed-off-by: Xavier Hernandez <xhernandez@datalab.es>
Reviewed-on: http://review.gluster.org/8892
Reviewed-by: Emmanuel Dreyfus <manu@netbsd.org>
Tested-by: Emmanuel Dreyfus <manu@netbsd.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Dan Lambright <dlambrig@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'tests/basic/ec/statedump.t')
| -rw-r--r-- | tests/basic/ec/statedump.t | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tests/basic/ec/statedump.t b/tests/basic/ec/statedump.t index 6881c7f8c2..8d311ec3d6 100644 --- a/tests/basic/ec/statedump.t +++ b/tests/basic/ec/statedump.t @@ -8,7 +8,9 @@ cleanup TEST glusterd TEST pidof glusterd TEST $CLI volume create $V0 redundancy 1 $H0:$B0/${V0}{0..2} +EXPECT "Created" volinfo_field $V0 'Status' TEST $CLI volume start $V0 +EXPECT_WITHIN $PROCESS_UP_TIMEOUT "Started" volinfo_field $V0 'Status' TEST $GFS --volfile-id=/$V0 --volfile-server=$H0 $M0 EXPECT_WITHIN $CHILD_UP_TIMEOUT "3" ec_child_up_count $V0 0 |