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

`lfs check` documentation is misleading

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.13.0
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      `lfs check` command documentation and arguments are misleading,
      it is used to check targets, not servers, as documentation states (or am I wrong?).

       

      I propose to change man page and add new arguments:

      • mdts
      • all

      Proposing patch is in attachment.

      Attachments

        Issue Links

          Activity

            [LU-10966] `lfs check` documentation is misleading

            "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49493
            Subject: LU-10966 utils: Fix `lfs check` documentation and arguments
            Project: fs/lustre-release
            Branch: b2_12
            Current Patch Set: 1
            Commit: 61bcd9c7a0a47fbfed3c14a1ab4d00b85e065687

            gerrit Gerrit Updater added a comment - "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49493 Subject: LU-10966 utils: Fix `lfs check` documentation and arguments Project: fs/lustre-release Branch: b2_12 Current Patch Set: 1 Commit: 61bcd9c7a0a47fbfed3c14a1ab4d00b85e065687

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33775/
            Subject: LU-10966 utils: Fix `lfs check` documentation and arguments
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 0652c828c93f266b9999883b1b2967bfaac11c89

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33775/ Subject: LU-10966 utils: Fix `lfs check` documentation and arguments Project: fs/lustre-release Branch: master Current Patch Set: Commit: 0652c828c93f266b9999883b1b2967bfaac11c89

            James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33775
            Subject: LU-10966 utils: Fix `lfs check` documentation and arguments
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 54cb5f6bafe36552811398367e0a957da99931cb

            gerrit Gerrit Updater added a comment - James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33775 Subject: LU-10966 utils: Fix `lfs check` documentation and arguments Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 54cb5f6bafe36552811398367e0a957da99931cb

            Minh Diep, thank you, I'll try it on my next contribution.

            melikov George Melikov (Inactive) added a comment - Minh Diep, thank you, I'll try it on my next contribution.
            mdiep Minh Diep added a comment -

            melikov, you should be able to do that now. we have added you to the list.

            mdiep Minh Diep added a comment - melikov , you should be able to do that now. we have added you to the list.
            melikov George Melikov (Inactive) added a comment - - edited

            @simmonsja Thank you!

            @mdiep I've used exactly this page, this command didn't work for me too

            git push ssh://gmelikov@review.whamcloud.com:29418/fs/lustre-release HEAD:refs/for/master
            

             

            melikov George Melikov (Inactive) added a comment - - edited @simmonsja Thank you! @mdiep I've used exactly this page, this command didn't work for me too git push ssh://gmelikov@review.whamcloud.com:29418/fs/lustre-release HEAD:refs/ for /master  

            Note your fix has been merged into https://review.whamcloud.com/#/c/31618

            simmonsja James A Simmons added a comment - Note your fix has been merged into  https://review.whamcloud.com/#/c/31618
            mdiep Minh Diep added a comment -

            Please see https://wiki.hpdd.intel.com/display/PUB/Using+Gerrit for how to contribute patches

            mdiep Minh Diep added a comment - Please see https://wiki.hpdd.intel.com/display/PUB/Using+Gerrit  for how to contribute patches

            Unfortunately, I can't push patch, is something wrong with command?

            git push ssh://gmelikov@review.whamcloud.com:29418/fs/lustre-release master
            
            fatal: Upload denied for project 'fs/lustre-release'
            fatal: Could not read from remote repository.
            
            Please make sure you have the correct access rights
            and the repository exists.
            
            melikov George Melikov (Inactive) added a comment - Unfortunately, I can't push patch, is something wrong with command? git push ssh: //gmelikov@review.whamcloud.com:29418/fs/lustre-release master fatal: Upload denied for project 'fs/lustre-release' fatal: Could not read from remote repository. Please make sure you have the correct access rights and the repository exists.

            People

              simmonsja James A Simmons
              melikov George Melikov (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: