[Bugs] [Bug 1672480] Bugs Test Module tests failing on s390x

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1672480

abhays <abhaysingh1722 at yahoo.in> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(abhaysingh1722 at ya |
                   |hoo.in)                     |



--- Comment #12 from abhays <abhaysingh1722 at yahoo.in> ---
(In reply to Nithya Balachandran from comment #9)
> The diffs for possible fixes for bug-1161311.t and bug-1193636.t are as
> follows. Please make the changes on your setup and let me know if it works.
> 
> 
> [root at rhgs313-6 tests]# git diff bugs/distribute/bug-1161311.t
> diff --git a/tests/bugs/distribute/bug-1161311.t
> b/tests/bugs/distribute/bug-1161311.t
> index c52c69b..3dc45a4 100755
> --- a/tests/bugs/distribute/bug-1161311.t
> +++ b/tests/bugs/distribute/bug-1161311.t
> @@ -82,6 +82,8 @@ for i in {1..10}; do
>    cat /tmp/FILE2 >> $M0/dir1/FILE2
>  done
>  
> +brick_loc=$(get_backend_paths $M0/dir1/FILE2)
> +
>  #dd if=/dev/urandom of=$M0/dir1/FILE2 bs=64k count=10240
>  
>  # Rename the file to create a linkto, for rebalance to
> @@ -99,7 +101,7 @@ TEST $CLI volume rebalance $V0 start force
>  
>  # Wait for FILE to get the sticky bit on, so that file is under
>  # active rebalance, before creating the links
> -TEST checksticky $B0/${V0}3/dir1/FILE1
> +TEST checksticky $brick_loc
>  
>  # Create the links
>  ## FILE3 FILE5 FILE7 have hashes, c8c91469 566d26ce 22ce7eba
> @@ -120,7 +122,7 @@ cd /
>  
>  # Ideally for this test to have done its job, the file should still be
>  # under migration, so check the sticky bit again
> -TEST checksticky $B0/${V0}3/dir1/FILE1
> +TEST checksticky $brick_loc
>  
>  # Wait for rebalance to complete
>  EXPECT_WITHIN $REBALANCE_TIMEOUT "completed" rebalance_status_field $V0
> 
> 
> 
> [root at rhgs313-6 tests]# git diff ./bugs/distribute/bug-1193636.t
> diff --git a/tests/bugs/distribute/bug-1193636.t
> b/tests/bugs/distribute/bug-1193636.t
> index ccde02e..6ffa2d9 100644
> --- a/tests/bugs/distribute/bug-1193636.t
> +++ b/tests/bugs/distribute/bug-1193636.t
> @@ -37,6 +37,8 @@ TEST mkdir $M0/dir1
>  # Create a large file (1GB), so that rebalance takes time
>  dd if=/dev/zero of=$M0/dir1/FILE2 bs=64k count=10240
>  
> +brick_loc=$(get_backend_paths $M0/dir1/FILE2)
> +
>  # Rename the file to create a linkto, for rebalance to
>  # act on the file
>  TEST mv $M0/dir1/FILE2 $M0/dir1/FILE1
> @@ -45,7 +47,7 @@ build_tester $(dirname $0)/bug-1193636.c
>  
>  TEST $CLI volume rebalance $V0 start force
>  
> -TEST checksticky $B0/${V0}3/dir1/FILE1
> +TEST checksticky $brick_loc
>  
>  TEST setfattr -n "user.test1" -v "test1" $M0/dir1/FILE1
>  TEST setfattr -n "user.test2" -v "test1" $M0/dir1/FILE1



(In reply to Nithya Balachandran from comment #10)
> Possible fix for ./tests/bugs/posix/bug-1619720.t:
> 
> 
> [root at rhgs313-6 tests]# git diff  bugs/posix/bug-1619720.t
> diff --git a/tests/bugs/posix/bug-1619720.t b/tests/bugs/posix/bug-1619720.t
> index bfd304d..8584476 100755
> --- a/tests/bugs/posix/bug-1619720.t
> +++ b/tests/bugs/posix/bug-1619720.t
> @@ -1,6 +1,7 @@
>  #!/bin/bash
>  
>  . $(dirname $0)/../../include.rc
> +. $(dirname $0)/../../volume.rc
>  . $(dirname $0)/../../dht.rc
>  
>  cleanup;
> @@ -35,7 +36,8 @@ TEST mkdir $M0/tmp
>  # file-2 will hash to the other subvol
>  
>  TEST touch $M0/tmp/file-2
> -pgfid_xattr_name=$(getfattr -m "trusted.pgfid.*" $B0/${V0}1/tmp/file-2 |
> grep "trusted.pgfid")
> +loc_2=$(get_backend_paths $M0/tmp/file-2)
> +pgfid_xattr_name=$(getfattr -m "trusted.pgfid.*" $loc_2 | grep
> "trusted.pgfid")
>  echo $pgfid_xattr_name

Thanks for the reply @Nithya. 
Unfortunately the above changes for
bugs/distribute/bug-1161311,bugs/distribute/bug-1193636.t and
bugs/posix/bug-1619720.t do not work.
PFA the logs for the same.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.