Hello, I've found that `lfs check` command documentation and arguments are misleading, it is used to check targets, not servers, as documentation states (or am I wrong?).
I didn't found the better place for patches or discussions, so I propose a patch here. I'll be glad if someone can give me the right direction for such cases. From 2aaf4fa4a12ce38d283725410f4f7d0f3b1488ab Mon Sep 17 00:00:00 2001 From: George Melikov <m...@gmelikov.ru> Date: Fri, 20 Apr 2018 15:27:00 +0300 Subject: [PATCH] Fix `lfs check` documentation and arguments --- lustre/doc/lfs.1 | 8 +++++--- lustre/utils/lfs.c | 4 ++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/lustre/doc/lfs.1 b/lustre/doc/lfs.1 index 29cd139519..714785fa63 100644 --- a/lustre/doc/lfs.1 +++ b/lustre/doc/lfs.1 @@ -7,7 +7,7 @@ lfs \- client utility for Lustre-specific file layout and other attributes .br .B lfs changelog_clear <mdtname> <id> <endrec> .br -.B lfs check <mds|osts|servers> +.B lfs check <mds|mdts|osts|all|servers> .br .B lfs data_version [-n] \fB<filename>\fR .br @@ -151,8 +151,10 @@ interest to a particular consumer <id>, potentially allowing the MDT to free up disk space. An <endrec> of 0 indicates the current last record. Changelog consumers must be registered on the MDT node using \fBlctl\fR. .TP -.B check -Display the status of MDS or OSTs (as specified in the command) or all the servers (MDS and OSTs) +.B check <mds|mdts|osts|all|servers> +Check and display the status of MDTs or OSTs (as specified in the command) +or all (MDTs and OSTs). \fBmds\fR and \fBservers\fR arguments are equal to +\fBmdts\fR and \fBall\fR, respectively, and remain for backward compatibility only. .TP .B data_version [-n] <filename> Display current version of file data. If -n is specified, data version is read diff --git a/lustre/utils/lfs.c b/lustre/utils/lfs.c index 4dcb8f31a1..a7a9de9e73 100644 --- a/lustre/utils/lfs.c +++ b/lustre/utils/lfs.c @@ -5152,9 +5152,9 @@ static int lfs_check(int argc, char **argv) if (strcmp(argv[1], "osts") == 0) { strcpy(obd_types[0], "osc"); - } else if (strcmp(argv[1], "mds") == 0) { + } else if (strcmp(argv[1], "mds") == 0 || strcmp(argv[1], "mdts") == 0) { strcpy(obd_types[0], "mdc"); - } else if (strcmp(argv[1], "servers") == 0) { + } else if (strcmp(argv[1], "servers") == 0 || strcmp(argv[1], "all") == 0) { num_types = 2; strcpy(obd_types[0], "osc"); strcpy(obd_types[1], "mdc"); -- 2.14.2 ____________________________________ Sincerely, George Melikov, Tel. 7-915-278-39-36 Skype: georgemelikov С наилучшими пожеланиями, Георгий Меликов, m...@gmelikov.ru Моб: +7 9152783936 Skype: georgemelikov _______________________________________________ lustre-discuss mailing list lustre-discuss@lists.lustre.org http://lists.lustre.org/listinfo.cgi/lustre-discuss-lustre.org