[LU-2886] create local files using local_storage library

• Type: Bug
• Resolution: Fixed
• Priority: Blocker
• Fix Version/s: Lustre 2.4.0
• Affects Version/s: Lustre 2.4.0
• Labels: MB
• Severity: 3
• Assignee: Mikhail Pershin
• Reporter: Mikhail Pershin

Description

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

Issue Links

• is related to: LU-838 "lfs path2fid /mnt/lustre" (ROOT) returns inode number

Comments

[Mikhail Pershin]

Old fixed FIDs will not be reused because FID_SEQ_LOCAL_FILE is not used by local storage at all. Strictly speaking this sequence should be used only for nameless objects created by dt_object_find_or_create(). There is no need in compatibility efforts in this area.

[Jodi Levi (Inactive)]

Patches landed to master.

[nasf (Inactive)]

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.

[nasf (Inactive)]

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?

[Peter Jones]

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

[Peter Jones]

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

[Andreas Dilger]

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

[Jodi Levi (Inactive)]

http://review.whamcloud.com/#change,6107

[Andreas Dilger]

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

[Peter Jones]

Landed for 2.4

[John Hammond]

See http://review.whamcloud.com/#change,4682.