[LU-13841] error messages need additional explanation for lctl pcc commands

• Type: Bug
• Resolution: Fixed
• Priority: Trivial
• Affects Version/s: Lustre 2.14.0
• Environment: Virtual machine
• Severity: 3
• Assignee: WC Triage
• Reporter: Vikentsi Lapa

Description

 

From user  perspective:

• different cases reports same error. 
• error message itself does not explain what File exists, so it should be documented or changed. Also error message assumed that user knows PCC internals to understand it.

 

From my point of view this can fixed by added some notes or explanation to documentation, or change error message itself, like "cache already enabled for ..."

 

Steps to reproduce:

Directory  for cache is created and exists.

1. ls /mnt/pcc2

#  lctl pcc add /mnt/lustre /mnt/pcc2 --param "projid={100} rwid=2"

Try to add pcc cache multiple times with same rwid

#  lctl pcc add /mnt/lustre /mnt/pcc2 --param "projid={100} rwid=2"

lctl pcc pcc: error: setting llite.lustre-ffff8b405721f800.pcc='add /mnt/pcc2 projid={100} rwid=2': File exists (17)

 

Try to add pcc cache with same path to /mtn/pcc2 but with  with different rwid

[#  lctl pcc add /mnt/lustre /mnt/pcc2 --param "projid={100} rwid=5"

lctl pcc pcc: error: setting llite.lustre-ffff8b405721f800.pcc='add /mnt/pcc2 projid={100} rwid=5*': File exists (17)*

 

Use incorrect arguments: to get path with numbers

1. lctl pcc add /mnt/lustre pccro --param projid={100} rwid=2
   add: must specify mount path and PCC path 3:6
   Add a PCC backend to a client.
   usage: lctl pcc add <mntpath> <pccpath> [--param|-p <param>]
   mntpath: Lustre mount point.
   pccpath: Path of the PCC backend.
   param: Setting parameters for PCC backend.

 What is "PCC path 3:6" about? Similar to lctl pcc del

lctl pcc del /mnt/pcc2
del: must speficy mount path and PCC path 1:2
Delete the specified PCC backend on a client.
usage: clt pcc del <mntpath> <pccpath>

 

Typo in error mesage:

del: must speficy mount path and PCC path 1:2  -> specify

 

Incorrect number of arguments in help output:

1. lctl pcc list
   list: require 2 arguments
   List all PCC backends on a client.
   usage: lctl pcc list <mntpath>**Command output shows 2 arguments in help but only one is required.

 

Issue Links

• is related to: LU-10499 Readonly Persistent Client Cache support

Comments

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52246
Subject: LU-13841 clio: remove cp_type from vvp, mdc, ll
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a8f9c58520158b222936a4593af8e19da5b0f0ae

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52245
Subject: LU-13841 osc: remove cp_type in osc
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 577fde0c95f0276b76fcf44040b4aa72ae170a7c

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52244
Subject: LU-13841 osc: change asserts to use OTP type
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e6f6197e37cff3d05f63650de5b9ba55cadcd434

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52243
Subject: LU-13841 clio: remove cl_page allocation for DIO
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3d3a345e8deaa3edca3b549abc416ec0cd6aa9b0

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52242
Subject: LU-13841 osc: flip the switch
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6bba631b7a2511ee97d909ee759dff47cc749080

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52241
Subject: LU-13841 osc: make transfer_page_init called for DIO
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4c8c701c29885f537b950e368c82e799a848c01d

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52240
Subject: LU-13841 clio: add cdp_count
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d964e0636036f3f60ba48066df8cec9ecb010d94

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52239
Subject: LU-13841 osc: make transfer_page_init take an otp
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8a47d61b4cd9ef1a6ad9084d77ebb3eedce1f523

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52182
Subject: LU-13841 osc: simplify osc_dio_page_submit
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 06011474f68d4985ca5c97f21897b3491b1af5a2

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52167
Subject: LU-13841 osc: add osc_dio_completion
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0ef54bbb5d39332edb9c9e9bb539674a50505f48

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52165
Subject: LU-13841 clio: add osc_dio_pages_init
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0ff20153b02afd9665b7a88e95f6f9f8d80de465