Skip to content

Commit

Permalink
Fix removing VM from obsolete_vms
Browse files Browse the repository at this point in the history
The domain-delete event gets string as a 'vm' argument, so use it
directly instead of accessing its 'name' attribute.

Fixes: 25f1be1 "Remove VM from obsolete_vms list if template is deleted"
  • Loading branch information
marmarek committed Jul 9, 2024
1 parent 66e5cfb commit f11af35
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions qui/tray/updates.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,8 +170,8 @@ def domain_removed(self, _submitter, _event, vm, *_args, **_kwargs):
if vm in self.vms_needing_update:
self.vms_needing_update.remove(vm)
self.update_indicator_state()
if vm.name in self.obsolete_vms:
self.obsolete_vms.remove(vm.name)
if vm in self.obsolete_vms:
self.obsolete_vms.remove(vm)
self.update_indicator_state()

def feature_unset(self, vm, event, feature, **_kwargs):
Expand Down

0 comments on commit f11af35

Please sign in to comment.