diff options
author | Jeff Darcy <jdarcy@redhat.com> | 2015-03-30 13:22:45 -0400 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-04-01 01:49:37 -0700 |
commit | 6d083f47e1399c58da02836f33fdb7d9c31fd689 (patch) | |
tree | 638aea290e1d3cae4dbe3314e28cf8adcc915b06 /ChangeLog | |
parent | ef5c3182224940f76d9290c1c4d9cc8e78fbdb83 (diff) |
tests: fix volume_exists to be used from EXPECT_WITHIN
Fixes the spurious volume-snapshot-clone.t regression failures. In
brief, the problem is that the script wasn't waiting for config commands
to complete, and would *sometimes* query the status of a volume while
that volume was still being deleted.
It turns out that "!" doesn't work properly from EXPECT_WITHIN, so there
was a choice between changing that or changing volume_exists. This
seemed less risky. Because of code duplication, two instances of the
function had to be changed, and the other caller (volume-snapshot.t) did
too.
Change-Id: I766d4dc7c5b11038ede8e45d9d1f29cd02a622a0
BUG: 1163543
Signed-off-by: Jeff Darcy <jdarcy@redhat.com>
Reviewed-on: http://review.gluster.org/10053
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'ChangeLog')
0 files changed, 0 insertions, 0 deletions