[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: solution: |
| 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 |
| 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 |