test: CLEANUP make test clearer by using "fail" as argument

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit ce85112d353b30a615093643d29410432d69b4cd)

diff --git a/src/test/erasure-code/test-erasure-eio.sh b/src/test/erasure-code/test-erasure-eio.sh
index 2d51bc5057aa125753a85516ad4771bd4d08efd4..aa2b2c3d9b9309d8a2d6491a0854fb99a00330ab 100755 (executable)
--- a/src/test/erasure-code/test-erasure-eio.sh
+++ b/src/test/erasure-code/test-erasure-eio.sh
@@ -87,12 +87,12 @@ function rados_get() {
     local dir=$1
     local poolname=$2
     local objname=${3:-SOMETHING}
-    local expect=${4:-0}
+    local expect=${4:-ok}
 
     #
     # Expect a failure to get object
     #
-    if [ $expect = "1" ];
+    if [ $expect = "fail" ];
     then
         ! rados --pool $poolname get $objname $dir/COPY
         return
@@ -116,7 +116,7 @@ function rados_put_get() {
     #
     rados_put $dir $poolname $objname || return 1
     # We can read even though caller injected read error on one of the shards
-    rados_get $dir $poolname $objname 0 || return 1
+    rados_get $dir $poolname $objname || return 1
 
     if [ -n "$recovery" ];
     then
@@ -130,7 +130,7 @@ function rados_put_get() {
         ceph osd out ${initial_osds[$last]} || return 1
         ! get_osds $poolname $objname | grep '\<'${initial_osds[$last]}'\>' || return 1
         # This will fail since one shard is out and one shard has injected read error
-        rados_get $dir $poolname $objname 1 || return 1
+        rados_get $dir $poolname $objname fail || return 1
         ceph osd in ${initial_osds[$last]} || return 1
     fi
 
@@ -171,7 +171,7 @@ function rados_get_data_eio() {
     shard_id=$(expr $shard_id + 1)
     inject_eio $objname $dir $shard_id || return 1
     # Now 2 out of 3 shards get EIO, so should fail
-    rados_get $dir $poolname $objname 1 || return 1
+    rados_get $dir $poolname $objname fail || return 1
 }
 
 # Change the size of speificied shard
@@ -221,12 +221,12 @@ function rados_get_data_bad_size() {
     #
     set_size $objname $dir $shard_id $bytes $mode || return 1
 
-    rados_get $dir $poolname $objname 0 || return 1
+    rados_get $dir $poolname $objname || return 1
 
     # Leave objname and modify another shard
     shard_id=$(expr $shard_id + 1)
     set_size $objname $dir $shard_id $bytes $mode || return 1
-    rados_get $dir $poolname $objname 1 || return 1
+    rados_get $dir $poolname $objname fail || return 1
 }
 
 #