Giuseppe Scrivano <gscrivan@xxxxxxxxxx> writes: > Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1219427 > > Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx> > --- > virtManager/delete.py | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/virtManager/delete.py b/virtManager/delete.py > index 2addcfa..a7d4ec2 100644 > --- a/virtManager/delete.py > +++ b/virtManager/delete.py > @@ -236,7 +236,11 @@ def populate_storage_list(storage_list, vm, conn): > if disk.source_pool: > try: > pool = conn.get_pool(disk.source_pool) > + if pool is None: > + return disk.path > vol = pool.get_volume(disk.path) > + if vol is None: > + return disk.path > return vol.get_target_path() > except KeyError: > return disk.path looking into the root cause of it.. commit 5357b91402fb7a8a73921216926908c08f6ad99d changed the semantic of conn.get_(vm|pool|interface|nodedev|net), to return None instead of raising KeyError. Should we perhaps restore the previous behavior? Thanks, Giuseppe _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
