Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx> writes: > https://bugzilla.redhat.com/show_bug.cgi?id=1108922 > > Signed-off-by: Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx> > --- > virtinst/diskbackend.py | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/virtinst/diskbackend.py b/virtinst/diskbackend.py > index 5f72d00..5725e1f 100644 > --- a/virtinst/diskbackend.py > +++ b/virtinst/diskbackend.py > @@ -65,7 +65,8 @@ def check_if_path_managed(conn, path): > def lookup_vol_by_path(): > try: > vol = conn.storageVolLookupByPath(path) > - vol.info() > + if os.path.exists(vol.path()): > + vol.info() I think that "check_if_path_managed" is used also in the case of a remote connection so os.path.exists is not correct as it checks for a local file. > return vol, None > except libvirt.libvirtError, e: > if (hasattr(libvirt, "VIR_ERR_NO_STORAGE_VOL") I have tried several times by using a similar command line as in the bug report but I wasn't able to reproduce the problem here (both virt-install-1.0.1-3.fc20.noarch and upstream version), what arguments did you feed to virt-install to get this failure? Thanks, Giuseppe _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
