Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-3514

sanity tests 27c, 77c, 77e, 77h are skipped when previous dependent test is not ran

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.5.0
    • Lustre 2.5.0
    • 3
    • 8845

    Description

      sanity tests 27c, 77c, 77e, 77h are skipped when previous dependent test is not ran:

      sanity.sh:
      
      test_27c() {
             [ -f $DIR/d27/f01 ] || skip "test_27b not run" && return
      ...
      }
      
      test_77c() {
              [ ! -f $DIR/f77b ] && skip "requires 77b - skipping" && return
      

      Attachments

        Issue Links

          Activity

            [LU-3514] sanity tests 27c, 77c, 77e, 77h are skipped when previous dependent test is not ran
            jamesanunez James Nunez (Inactive) made changes -
            Link New: This issue is related to LU-2524 [ LU-2524 ]
            jamesanunez James Nunez (Inactive) made changes -
            Status Original: Resolved [ 5 ] New: Closed [ 6 ]
            jamesanunez James Nunez (Inactive) made changes -
            Fix Version/s New: Lustre 2.5.0 [ 10295 ]
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]

            Landed to master

            jamesanunez James Nunez (Inactive) added a comment - Landed to master
            yujian Jian Yu added a comment -

            Would you please comment on if the tests in the proposed patch should be combined?

            IMHO, these sub-tests can be combined. I just reviewed the patch.

            yujian Jian Yu added a comment - Would you please comment on if the tests in the proposed patch should be combined? IMHO, these sub-tests can be combined. I just reviewed the patch.

            Yu, Jian

            Would you please comment on if the tests in the proposed patch should be combined?

            Thank you,
            James

            jamesanunez James Nunez (Inactive) added a comment - Yu, Jian Would you please comment on if the tests in the proposed patch should be combined? Thank you, James
            jamesanunez James Nunez (Inactive) made changes -
            Assignee Original: WC Triage [ wc-triage ] New: James Nunez [ jamesanunez ]

            http://review.whamcloud.com/#/c/6784

            This patch combines the following sanity tests:
            tests 27b, 27c to test 27b
            tests 77b, 77c to test 77b
            tests 77d, 77e to test 77d
            tests 77g, 77h to test 77g

            egryaznova Elena Gryaznova added a comment - http://review.whamcloud.com/#/c/6784 This patch combines the following sanity tests: tests 27b, 27c to test 27b tests 77b, 77c to test 77b tests 77d, 77e to test 77d tests 77g, 77h to test 77g
            egryaznova Elena Gryaznova created issue -

            People

              jamesanunez James Nunez (Inactive)
              egryaznova Elena Gryaznova
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: