summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEmmanuel Dreyfus <manu@netbsd.org>2015-01-08 06:22:17 +0100
committerVijay Bellur <vbellur@redhat.com>2015-01-09 09:54:55 -0800
commit9408dc7b416ca80b3b8d8ecae2ef75c7e9cd21cd (patch)
tree1b7d05bb6e0ed8962b523a87469bce8ef69f13d0
parenteaf7164ff824ad4d69cf225e49c54fc473b300b8 (diff)
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 966972e5443..ee0afaf9d4e 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