On Tuesday 10 November 2015 12:06:21 Richard W.M. Jones wrote: > When a disk image uses LVM thinp (thin provisioning), the guestfs_lvs > API would return the thinp pools. This confused other APIs because > thinp pools don't have corresponding /dev/VG/LV device nodes. > > Filter the LVs that are returned using "lv_role=public". > > Thanks: Fabian Deutsch > --- > daemon/lvm.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/daemon/lvm.c b/daemon/lvm.c > index d989986..6e201e3 100644 > --- a/daemon/lvm.c > +++ b/daemon/lvm.c > @@ -148,7 +148,9 @@ do_lvs (void) > > r = command (&out, &err, > str_lvm, "lvs", > - "-o", "vg_name,lv_name", "--noheadings", > + "-o", "vg_name,lv_name", > + "-S", "lv_role=public", > + "--noheadings", > "--separator", "/", NULL); > if (r == -1) { > reply_with_error ("%s", err); Should the additional parameter passed also when calling `lvs` in parse_command_line_lvs (which implements lvs_full)?