Details

    • Bug
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      the patch https://review.whamcloud.com/50240 demonstrates that number of tests don't really hit fail_loc they set, meaning they are useless:

      sanityn: 47a, 47b, 47d, 47e, 47g, 92
      sanity-quota: 7d
      recovery-small: 19c, 27
      replay-dual: 12, 13
      replay-single: 47, 57, 58a, 61a, 61b, 61c, 67b, 84a, 103
      sanity-flr: 36a, 200b
      sanity: 17h, 60c, 117, 118n, 119d, 123b, 152, 221, 222b, 223, 239b, 258a, 258b, 273c, 398f, 399b, 419

      Attachments

        Issue Links

          Activity

            [LU-16636] some tests don't hit fail_loc

            There are also several other OBD_FAIL cases listed in LU-10363 that are set on the client and checked on the server, but I haven't looked if they overlap with the test cases here.

            adilger Andreas Dilger added a comment - There are also several other OBD_FAIL cases listed in LU-10363 that are set on the client and checked on the server, but I haven't looked if they overlap with the test cases here.
            adilger Andreas Dilger added a comment - - edited

            It looks like patch https://review.whamcloud.com/56113 "LU-12567 ptlrpc: handle reply and resend reorder" is fixing sanity.sh test_224 which was setting fail_loc=OBD_FAIL_PTLRPC_CLIENT_BULK_CB=0x515 on the client and checking it in the OST target code.

            adilger Andreas Dilger added a comment - - edited It looks like patch https://review.whamcloud.com/56113 " LU-12567 ptlrpc: handle reply and resend reorder " is fixing sanity.sh test_224 which was setting fail_loc=OBD_FAIL_PTLRPC_CLIENT_BULK_CB=0x515 on the client and checking it in the OST target code.

            The sanity.sh test_119d subtest is removed via patch https://review.whamcloud.com/50731 "LU-13706 tests: remove test 119d".

            adilger Andreas Dilger added a comment - The sanity.sh test_119d subtest is removed via patch https://review.whamcloud.com/50731 " LU-13706 tests: remove test 119d ".

            People

              bzzz Alex Zhuravlev
              bzzz Alex Zhuravlev
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: