search for: 42260af

...y are partitioned) --- tests/md/Makefile.am | 1 + tests/md/test-list-filesystems2.sh | 79 ++++++++++++++++++++++++++++++++++++++ 2 files changed, 80 insertions(+) create mode 100755 tests/md/test-list-filesystems2.sh diff --git a/tests/md/Makefile.am b/tests/md/Makefile.am index 42260af..9be9ca4 100644 --- a/tests/md/Makefile.am +++ b/tests/md/Makefile.am @@ -21,6 +21,7 @@ TESTS = \ test-inspect-fstab.sh \ test-inspect-fstab-md.sh \ test-list-filesystems.sh \ + test-list-filesystems2.sh \ test-list-md-devices.sh \ test-lvm-on-md-device.sh \ test-md-and-lvm-devices.sh \...

Initial discussion is here: https://www.redhat.com/archives/libguestfs/2018-January/msg00188.html. v2 was posted here: https://www.redhat.com/archives/libguestfs/2018-January/msg00246.html. v3 comparing to v2 is just a rebase with slightly changed commits comments.

Instead of parsing 'parted' output OCaml implementation relies on the following facts: 1. The function is applicable for MBR partitions only (as noted in documentation and as function name suggests). 2. An attempt to call the function for non-MBR partition fails with "part_get_mbr_part_type can only be used on MBR Partitions" error and NULL is returned. 3. MBR partition table

Before I rush to change something I request your comments on the subject. Let me know what do you think and if it does make sense. The issue: guesfs_list_filesystems is inconsistent in its output. For, example, it filters out partitioned physical devices but doesn't do the same for MD devices. More over, according to its name and API documentation guestfs_list_filesystem should return