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/+/52249
            Subject: LU-13841 clio: remove type from cl_page_find
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 71ec433e03ca2faad472ba941e7043fdefe2558f

            gerrit Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52249 Subject: LU-13841 clio: remove type from cl_page_find Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 71ec433e03ca2faad472ba941e7043fdefe2558f

            "Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52248
            Subject: LU-13841 clio: remove cp_type
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 0ea56126d6edf6fa552eefd18bb4072f74b32328

            gerrit Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52248 Subject: LU-13841 clio: remove cp_type Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 0ea56126d6edf6fa552eefd18bb4072f74b32328

            "Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52247
            Subject: LU-13841 lov: remove cp_type from lov
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: f179b6db9fcdc9dd018e5fe54515629e51d836b4

            gerrit Gerrit Updater added a comment - "Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52247 Subject: LU-13841 lov: remove cp_type from lov Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: f179b6db9fcdc9dd018e5fe54515629e51d836b4

            "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

            People

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

              Dates

                Created:
                Updated:
                Resolved: