[LU-2886] create local files using local_storage library

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

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) ret...

Comments

[John Hammond]

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

[Peter Jones]

Landed for 2.4

[Andreas Dilger]

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

[Jodi Levi (Inactive)]

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

[Andreas Dilger]

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

[Peter Jones]

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

[Peter Jones]

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

[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?

[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.

[Jodi Levi (Inactive)]

Patches landed to master.

[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.

[Andreas Dilger]

Reopened for http://review.whamcloud.com/6299 landing.

[Jodi Levi (Inactive)]

Patch landed to master