[LU-8962] lfs df prints nothing and exits with status 0 when given a non-lustre argument Created: 20/Dec/16 Updated: 23/Jul/23 Resolved: 22/Sep/21 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.14.0 |
| Fix Version/s: | Lustre 2.15.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | John Hammond | Assignee: | Arshad Hussain |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | easy | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
It also ignores everything after the first argument. t:lustre-release# lfs df /tmp
t:lustre-release# echo $?
0
t:lustre-release# lfs df /mnt/lustre XXX
UUID 1K-blocks Used Available Use% Mounted on
lustre-MDT0000_UUID 141752 19528 112864 15% /mnt/lustre[MDT:0]
lustre-OST0000_UUID 366744 29892 317380 9% /mnt/lustre[OST:0]
lustre-OST0001_UUID 366744 29892 317352 9% /mnt/lustre[OST:1]
filesystem_summary: 733488 59784 634732 9% /mnt/lustre
t:lustre-release# echo $?
0
t:lustre-release# lfs df XXX /mnt/lustre
error: invalid path 'XXX': No such file or directory
t:lustre-release# echo $?
2
|
| Comments |
| Comment by Andreas Dilger [ 14/Dec/17 ] |
|
"lfs df" should process all of the additional arguments (e.g. "lfs df /mnt/lustre /mnt/lustre2" should show status for both filesystems). If an error is hit during argument processing, the first such error should be saved and argument processing should continue, and the error returned at the end. |
| Comment by Andreas Dilger [ 05/Aug/20 ] |
|
It would also be useful to print a brief error message and non-zero return code if no argument is given and no Lustre filesystems are mounted on the node, like "lfs df: no Lustre filesystems found" and exit with "ENOENT = 2". |
| Comment by Gerrit Updater [ 22/Mar/21 ] |
|
Arshad Hussain (arshad.hussain@aeoncomputing.com) uploaded a new patch: https://review.whamcloud.com/42126 |
| Comment by Gerrit Updater [ 22/Sep/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42126/ |
| Comment by Peter Jones [ 22/Sep/21 ] |
|
Landed for 2.15 |