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

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.13.0
• Severity: 3
• Assignee: James A Simmons
• Reporter: George Melikov (Inactive)

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.

Issue Links

• is related to: LU-8066 Move lustre procfs handling to sysfs and debugfs.

Comments

[Gerrit Updater]

James Simmons ([e-mail]) 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

[George Melikov (Inactive)]

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

[Minh Diep]

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

[George Melikov (Inactive)]

@simmonsja Thank you!

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

git push ssh://[e-mail]:29418/fs/lustre-release HEAD:refs/for/master

 

[James A Simmons]

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

[Minh Diep]

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

[George Melikov (Inactive)]

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

git push ssh://[e-mail]: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.

[Joseph Gmitter (Inactive)]

The patch is for man page updates, therefore it should be moved to an LU ticket.