Details

    • Technical task
    • Resolution: Fixed
    • Critical
    • Lustre 2.5.0
    • None
    • 8966

    Description

      Add a test suite for HSM features

      Attachments

        Issue Links

          Activity

            [LU-3561] Add a sanity test for HSM

            Closing tickets per latest comments.

            jlevi Jodi Levi (Inactive) added a comment - Closing tickets per latest comments.

            This ticket was for initial landing and quick follow-on patches. So we can close it now

            jcl jacques-charles lafoucriere added a comment - This ticket was for initial landing and quick follow-on patches. So we can close it now
            bfaccini Bruno Faccini (Inactive) added a comment - - edited

            Primary patch/change #6913 for this ticket has been landed.

            Even if it claims LU-3561 in its Commit-msg, change #7437 has been submitted for LU-3726 (Major) ticket (to make sanity-hsm DNE aware), so seems to me it should better be tracked there.

            Change #7374 is only an update to sanity-hsm sub-tests exceptions, and was 1st submitted for LU-3815.

            So do we really need to leave this ticket open, or at least can we downgrade its priority, what do you all think ?

            bfaccini Bruno Faccini (Inactive) added a comment - - edited Primary patch/change #6913 for this ticket has been landed. Even if it claims LU-3561 in its Commit-msg, change #7437 has been submitted for LU-3726 (Major) ticket (to make sanity-hsm DNE aware), so seems to me it should better be tracked there. Change #7374 is only an update to sanity-hsm sub-tests exceptions, and was 1st submitted for LU-3815 . So do we really need to leave this ticket open, or at least can we downgrade its priority, what do you all think ?
            adilger Andreas Dilger added a comment - Two other patches still to be landed: http://review.whamcloud.com/7374 http://review.whamcloud.com/7437
            pjones Peter Jones added a comment -

            It seems like we should reopen this to track the other changsets in gerrit. There are three at the time of writing - http://review.whamcloud.com/#/q/status:open+message:3561,n,z

            pjones Peter Jones added a comment - It seems like we should reopen this to track the other changsets in gerrit. There are three at the time of writing - http://review.whamcloud.com/#/q/status:open+message:3561,n,z

            Change #6913 will need to rebase when all required ones land.

            bfaccini Bruno Faccini (Inactive) added a comment - Change #6913 will need to rebase when all required ones land.
            jhammond John Hammond added a comment -

            The tests added depend on the patches in this series and upon the patches in the copytool series.

            jhammond John Hammond added a comment - The tests added depend on the patches in this series and upon the patches in the copytool series.

            Patch auto-test failure is :

            Start copytool
            /usr/lib64/lustre/tests/sanity-hsm.sh: line 97: /usr/lib64/lustre/utils/lhsmtool_posix: No such file or directory
             sanity-hsm : @@@@@@ FAIL: Copytool not runnable: 0
            ....
            

            looks like master branch is missing the utils available under lustre-hsm branch.

            bfaccini Bruno Faccini (Inactive) added a comment - Patch auto-test failure is : Start copytool /usr/lib64/lustre/tests/sanity-hsm.sh: line 97: /usr/lib64/lustre/utils/lhsmtool_posix: No such file or directory sanity-hsm : @@@@@@ FAIL: Copytool not runnable: 0 .... looks like master branch is missing the utils available under lustre-hsm branch.

            Thank's Peter! Wow and what a change J-C ... [+2445, -104] in sanity-hsm script, enlargement is a too weak word.

            bfaccini Bruno Faccini (Inactive) added a comment - Thank's Peter! Wow and what a change J-C ... [+2445, -104] in sanity-hsm script, enlargement is a too weak word.
            pjones Peter Jones added a comment -

            Bruno

            There is a patch in gerrit - http://review.whamcloud.com/#/c/6913/

            Peter

            pjones Peter Jones added a comment - Bruno There is a patch in gerrit - http://review.whamcloud.com/#/c/6913/ Peter

            People

              bfaccini Bruno Faccini (Inactive)
              cealustre CEA
              Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: