[LU-12031] DoM/HSM: hsm_release fails after hsm_restore

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.16.0
• Affects Version/s: Lustre 2.12.0
• Labels: CEA DoM HSM
• Severity: 3
• Assignee: Mikhail Pershin
• Reporter: CEA

Description

 There is an issue when releasing a file striped with DoM after an hsm_restore.

To reproduce:

1) create a file with a 1st component on MDT:

lfs setstripe -E 1M -L mdt -E -1 -S 4M -c -1 /mnt/lustre/domfile

2) archive and release the file (requires HSM set up)
 

lfs hsm_archive /mnt/lustre/domfile
# (wait for archive to complete)
lfs hsm_release

3) restore the file

lfs hsm_restore /mnt/lustre/domfile
# or cat /mnt/lustre/domfile

4) release the file => FAILS 

lfs hsm_release /mnt/lustre/domfile

Cannot send HSM request (use of /mnt/lustre/domfile): Device or resource busy

 
It may be something wrong with the data version stored in hsm EA.

Issue Links

• is related to: LU-13515 prohibit DOM layout for VOLATILE file

Comments

[Mikhail Pershin]

there is nothing left on my side as well

[Sergey Cheremencev]

I've abandoned https://review.whamcloud.com/#/c/fs/lustre-release/+/47497/. Agree to close if there is no known issues after landing 47139.

[Peter Jones]

There is still an unhanded patch tracked under this ticket - https://review.whamcloud.com/#/c/fs/lustre-release/+/47497/ - but I'll defer to scherementsev as to whether that is needed...

[Nikitas Angelinas]

Peter, is there any remaining work for this issue or can it be closed?

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/47139/
Subject: LU-12031 mdt: explicit data version of DoM files
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: aae3289adb2bbc192870f195b78044484f717e16

[Gerrit Updater]

"Sergey Cheremencev <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/47497
Subject: LU-12031 mdt: proof of concept
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b05ecdbcc682c1d2ef290110bd052bc3e0f2e61a

[Andreas Dilger]

Rather than storing yet another xattr on the DoM inode in this case (which might have issues with backup/restore, etc.), what about just not updating i_version on setxattr from HSM restore (or resetting it to the pre-update i_version)?

[Gerrit Updater]

"Mike Pershin <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/47139
Subject: LU-12031 mdt: explicit data version of DoM files
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5dbeabb034654f258162f05722477645e5f2fffe

[Andreas Dilger]

Ben - see LU-9961

[Ben Evans (Inactive)]

It would be nice if instead of all the playing around with temp files we could just restore to a stripe, and once completed mark it as primary.  We should also be able to restore all the other layout information as well and mark them as secondary.