[LU-4569] Posix copytool import a file using only its FID Created: 31/Jan/14  Updated: 11/Jun/14  Resolved: 20/May/14

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.6.0, Lustre 2.5.2

Type: Improvement Priority: Minor
Reporter: Henri Doreau (Inactive) Assignee: Bruno Faccini (Inactive)
Resolution: Fixed Votes: 0
Labels: HSM, patch

Rank (Obsolete): 12471

 Description   

The posix copytool imports files that are designated by their path in the backend, despite being itself the only one to know this path.

We propose a slight improvement to make the posix CT also accept a FID as import source parameter (from which the path can be rebuilt). This would allow easy "undeletion" of archived files:

lhsmtool_posix --import <fid> /mnt/lustre/undelete ...



 Comments   
Comment by Henri Doreau (Inactive) [ 31/Jan/14 ]

Patch at http://review.whamcloud.com/9075

Comment by Peter Jones [ 31/Jan/14 ]

Bruno

Could you please take care of this patch?

Thanks

Peter

Comment by Bruno Faccini (Inactive) [ 12/Feb/14 ]

The only few auto-tests errors are unrelated to this patch. Looks good to me.

Comment by Henri Doreau (Inactive) [ 23/Apr/14 ]

Any update on this?

Comment by Bruno Faccini (Inactive) [ 24/Apr/14 ]

Hello Henry, reviews are in progress so I think your patch should land soon now!

Comment by Peter Jones [ 24/Apr/14 ]

Landed for 2.6

Comment by James Nunez (Inactive) [ 28/Apr/14 ]

Patch for b2_5 at http://review.whamcloud.com/#/c/10132/

Comment by jacques-charles lafoucriere [ 29/Apr/14 ]

This patch missed the following use case:
1) create/archive a file1 with FID1
2) import a new file2 from FID1
3) restore file2
4) file1 and file2 share the same archive in backend so it will be corrupted after next archive of file1 or file2

solution:
(2) must be forbidden if FID1 exists in Lustre

Comment by Henri Doreau (Inactive) [ 29/Apr/14 ]

Thanks JC. I will submit a fix for master and an updated patch for 2.5.

Comment by Henri Doreau (Inactive) [ 01/May/14 ]

Patch for master: http://review.whamcloud.com/10185
The problem was already present when importing by path, the patch aims at fixing both cases.

Comment by Henri Doreau (Inactive) [ 03/May/14 ]

The check on import by path breaks sanity-hsm. Patchset #2 only performs the existence check when importing FIDs (more error-prone maybe?). Also, the extra access() call has a non-negligible impact, that might be undesirable when importing large directory tree which is typically what import-by-path would be useful for.

The problem is that sanity-hsm directly copies files to the archive without following the naming scheme used by the posix copytool. An option would be to copy those files into lustre, archiving them and removing them from lustre, but this is not totally equivalent (it involves many more components). Let me know if some solution is preferred.

Comment by Peter Jones [ 20/May/14 ]

Both patches landed to 2.6; will be landed to b2_5 separately

Generated at Sat Feb 10 01:43:54 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.