summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEmmanuel Dreyfus <manu@netbsd.org>2015-01-08 06:22:17 +0100
committergluster-ant <bugzilla-bot@gluster.org>2015-01-08 06:22:17 +0100
commit1132088a36bbb0fe9d661bf3e359c565929ec5b6 (patch)
treed06701ba65051780150c4b8bba0c00f64e787412
parentd78f3b5bfd9900c0b0ff692c79db2c33faf6ceaa (diff)
downloadglusterfs-1132088a36bbb0fe9d661bf3e359c565929ec5b6.tar.gz
glusterfs-1132088a36bbb0fe9d661bf3e359c565929ec5b6.tar.xz
glusterfs-1132088a36bbb0fe9d661bf3e359c565929ec5b6.zip
Fix bad shell calculation in tests/basiv/afr/self-heald.t
A shell variable calculation in tests/basiv/afr/self-heald.t resulted in spurious regression failure at test 67. This is wrong and always produce an error: $((`date +"%j"`))%2 This still fails because date +"%j" produces 008, which is considered an octal value: $((`date +"%j"` %2 )) This works: $((`date +"%j"|sed 's/^0*//'` % 2 )) BUG: 1129939 Change-Id: I5d0d27b7bb64ef7d56bafebe71aafe01eb2f39a7 Signed-off-by: Emmanuel Dreyfus <manu@netbsd.org> Reviewed-on: http://review.gluster.org/9414 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
-rw-r--r--tests/basic/afr/self-heald.t2
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/basic/afr/self-heald.t b/tests/basic/afr/self-heald.t
index 966972e544..ee0afaf9d4 100644
--- a/tests/basic/afr/self-heald.t
+++ b/tests/basic/afr/self-heald.t
@@ -51,7 +51,7 @@ TEST $CLI volume set $V0 cluster.eager-lock off
TEST $CLI volume start $V0
TEST glusterfs --volfile-id=/$V0 --volfile-server=$H0 $M0 --attribute-timeout=0 --entry-timeout=0
-decide_kill=$((`date +"%j"`))%2
+decide_kill=$((`date +"%j"|sed 's/^0*//'` % 2 ))
kill_multiple_bricks $V0 $H0 $B0
cd $M0