Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-13841

error messages need additional explanation for lctl pcc commands

Details

    • Bug
    • Resolution: Fixed
    • Trivial
    • None
    • Lustre 2.14.0
    • None
    • Virtual machine
    • 3
    • 9223372036854775807

    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.

       

      Attachments

        Issue Links

          Activity

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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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 Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            "Patrick Farrell <pfarrell@whamcloud.com>" 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

            gerrit Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" 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

            People

              wc-triage WC Triage
              vilapa Vikentsi Lapa
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: