Thread (6 messages) 6 messages, 3 authors, 2022-12-22

Re: [PATCH v2 1/2] btrfs-progs: fi show: Print missing device for a mounted file system

From: Nikolay Borisov <hidden>
Date: 2021-09-10 09:18:55


On 10.09.21 г. 12:15, Johannes Thumshirn wrote:
2021 10:33, Nikolay Borisov wrote:
quoted
Currently when a device is missing for a mounted filesystem the output
that is produced is unhelpful:

Label: none  uuid: 139ef309-021f-4b98-a3a8-ce230a83b1e2
	Total devices 2 FS bytes used 128.00KiB
	devid    1 size 5.00GiB used 1.26GiB path /dev/loop0
	*** Some devices missing

While the context which prints this is perfectly capable of showing
which device exactly is missing, like so:

Label: none  uuid: 4a85a40b-9b79-4bde-8e52-c65a550a176b
	Total devices 2 FS bytes used 128.00KiB
	devid    1 size 5.00GiB used 1.26GiB path /dev/loop0
	devid    2 size 0 used 0 path /dev/loop1 MISSING

This is a lot more usable output as it presents the user with the id
of the missing device and its path.

Signed-off-by: Nikolay Borisov <redacted>
---
Looks good,
Reviewed-by: Johannes Thumshirn <redacted>

but I think this also needs a patch for xfstests adjusting the filters.
It does, I had sent one, but I need to send v2 to adjust for the removed
***, ah this bike shedding ...
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help