Sandro Bonazzola has uploaded a new change for review. Change subject: ClientIF: avoid runtime errors at VM shutdown ......................................................................
ClientIF: avoid runtime errors at VM shutdown Avoids: RuntimeError: dictionary changed size during iteration Bug-Url: https://bugzilla.redhat.com/990985 Signed-off-by: Sandro Bonazzola <[email protected]> Change-Id: I4e39f777ce914b8b038505969767b6fd3ff9072f --- M vdsm/clientIF.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/99/17599/1 diff --git a/vdsm/clientIF.py b/vdsm/clientIF.py index c8b0ad3..014c4d0 100644 --- a/vdsm/clientIF.py +++ b/vdsm/clientIF.py @@ -122,7 +122,7 @@ self.domainVmIds[dom] = set([vmId]) def removeVmFromMonitoredDomains(self, vmId): - for dom in self.domainVmIds: + for dom in self.domainVmIds.keys(): #only take lock here to allow runVm to take the lock in-between with self.domainVmIdsLock: try: -- To view, visit http://gerrit.ovirt.org/17599 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4e39f777ce914b8b038505969767b6fd3ff9072f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
