Ayal Baron has submitted this change and it was merged. Change subject: libvirtvm: avoid concurrent VM changes during saveState ......................................................................
libvirtvm: avoid concurrent VM changes during saveState When multiple changes of the VM configuration are applied at the same time there's an high chance of saveState failing during a deepcopy for a RuntimeError exception (e.g. dictionary changed size during iteration). To avoid this problem the configuration changes and the saveState requests are now serialized. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=948940 Change-Id: Ic08b4073f5e3f5184baa5f1c7dd3ec5a148ff60b Signed-off-by: Federico Simoncelli <[email protected]> --- M vdsm/libvirtvm.py M vdsm/vm.py 2 files changed, 58 insertions(+), 33 deletions(-) Approvals: Ayal Baron: Looks good to me, approved Daniel Paikov: Verified -- To view, visit http://gerrit.ovirt.org/13624 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic08b4073f5e3f5184baa5f1c7dd3ec5a148ff60b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dafna Ron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Daniel Paikov <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
