[LU-10699] replace HSM actions llog with an index

• Status: Resolved
• Project: Lustre
• Type: Bug
• Priority: Minor
• Reporter: John Hammond
• Assignee: John Hammond
• Resolution: Won't Fix
• Labels: hsm
• Severity: 3

Issue Links

• is related to: LU-10383 deprecate HSM compound_id
• is related to: LU-10850 empty indexes don't iterate correctly...
• is related to: LU-10855 llog cleanups
• is related to: LU-10854 Remove start args from llog_cat_proce...

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31382
Subject: LU-10699 hsm: remove struct hsm_compat_data_cb
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 51cb00be305d4c15253e647d7ac1bbf8a891de87

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31385
Subject: LU-10699 hsm: simplify mdt_hsm_

{add,get}

_actions()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9f31bff9426f3f9c8b385a2a6bf577cce5ef63bf

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31758
Subject: LU-10699 hsm: add local object storage to MDTs
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bea8460d694ca3b2e120d70abdb98f2fddc71aeb

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31776
Subject: LU-10699 hsm: add HSM actions index
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e8a66c34148dc8e7cf898f957a319959ad79bebd

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31821
Subject: LU-10699 osd: add update methods to DT index API
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 805d782f9d62b8e454d087e73b8fbcbb990b76b2

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/31382/
Subject: LU-10699 hsm: remove struct hsm_compat_data_cb
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9321c8a3de6fe26b30a6eeac423e0407a8b1c564

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/31385/
Subject: LU-10699 hsm: simplify mdt_hsm_

{add,get}

_actions()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9bc0d837eb0cfc7e1d9ce702057761560d66434c

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/31758/
Subject: LU-10699 hsm: add local object storage to MDTs
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d8cd83addf0020b971c50f6928badf885590742c

[Nathan Rutman]

Is this trying to address LU-9540?

[John Hammond]

It's trying to address several things that are awkward or inefficient due to the use of a catalog for HSM actions. LU-13543 is the most recent example.

[Aurelien Degremont (Inactive)]

@John Hammond , could you explained why this solution was abandoned? Is there a fundamental flaw in its design?

[John Hammond]

> Is there a fundamental flaw in its design?

No. An index is better suited than an llog for the access patterns we need to support here. (Just like a RB tree is better than a linked list when ...). But I cannot spend time on this now due to other responsibilities.

Anyone who would like to pursue this should feel free to reopen the issue.