From fccd4ee5e846083096554a41fae831240fc72b1d Mon Sep 17 00:00:00 2001
From: "Richard W.M. Jones" <rjones@redhat.com>
Date: Tue, 10 Nov 2015 12:03:39 +0000
Subject: [PATCH] daemon: lvm: Only return public LVs from guestfs_lvs API
 (RHBZ#1278878).

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 d98998685..6e201e377 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);