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

create local files using local_storage library

Details

    • Bug
    • Resolution: Fixed
    • Blocker
    • Lustre 2.4.0
    • Lustre 2.4.0
    • 3
    • 6967

    Description

      Switch local file handling to local_storage library and stop using obsoleted md_local_file library. This is technical debt from Orion.

      Attachments

        Issue Links

          Activity

            [LU-2886] create local files using local_storage library

            Hi Tappro, since it is very near lustre-2.4 release and Russian seems in vacation. So I made a small patch (http://review.whamcloud.com/#change,6299) to fix the issue mentioned above as Andreas suggested, then it can be tested sooner.

            Please review it when you are back from vacation. Thanks.

            yong.fan nasf (Inactive) added a comment - Hi Tappro, since it is very near lustre-2.4 release and Russian seems in vacation. So I made a small patch ( http://review.whamcloud.com/#change,6299 ) to fix the issue mentioned above as Andreas suggested, then it can be tested sooner. Please review it when you are back from vacation. Thanks.
            yong.fan nasf (Inactive) added a comment - - edited

            Hi Tappro, for an old MDT upgrade to Lustre-2.4, there were some local files created with fixed FIDs {FID_SEQ_LOCAL_FILE, fixed_oid, 0}. How the new local_file_find_or_create() or local_index_find_or_create() can guarantee that the new create local files will not reuse these fixed FIDs?

            yong.fan nasf (Inactive) added a comment - - edited Hi Tappro, for an old MDT upgrade to Lustre-2.4, there were some local files created with fixed FIDs {FID_SEQ_LOCAL_FILE, fixed_oid, 0}. How the new local_file_find_or_create() or local_index_find_or_create() can guarantee that the new create local files will not reuse these fixed FIDs?
            pjones Peter Jones added a comment -

            Oops. It's the cleanup patch that just landed so setting back to a blocker

            pjones Peter Jones added a comment - Oops. It's the cleanup patch that just landed so setting back to a blocker
            pjones Peter Jones added a comment -

            Dropped priority as remaining work is cleanup rather than a blocker for 2.4

            pjones Peter Jones added a comment - Dropped priority as remaining work is cleanup rather than a blocker for 2.4

            http://review.whamcloud.com/6107 is a cleanup patch, not required to be landed for 2.4.0.

            adilger Andreas Dilger added a comment - http://review.whamcloud.com/6107 is a cleanup patch, not required to be landed for 2.4.0.
            jlevi Jodi Levi (Inactive) added a comment - http://review.whamcloud.com/#change,6107

            Reopen because http://review.whamcloud.com/6199 still needs to be landed.

            adilger Andreas Dilger added a comment - Reopen because http://review.whamcloud.com/6199 still needs to be landed.
            pjones Peter Jones added a comment -

            Landed for 2.4

            pjones Peter Jones added a comment - Landed for 2.4
            jhammond John Hammond added a comment - See http://review.whamcloud.com/#change,4682 .

            People

              tappro Mikhail Pershin
              tappro Mikhail Pershin
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: