Dan Kenigsberg has submitted this change and it was merged. Change subject: vm: safer handling of conf in restore ......................................................................
vm: safer handling of conf in restore In the restore state (aka dehibernation) path, there are a few places left on which the code mutates Vm.conf without the protection of the confLock. There is no obvious reason to have those mutations unguarded, so this patch adds the confLock protection. Change-Id: Ia46be9c779589b8c27395ced7904efb7de8cacc7 Backport-To: 3.6 Backport-To: 3.5 Related-To: https://bugzilla.redhat.com/1296936 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/51657 Reviewed-by: Martin Polednik <[email protected]> Continuous-Integration: Jenkins CI Reviewed-by: Milan Zamazal <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified Milan Zamazal: Looks good to me, but someone else must approve Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/51657 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia46be9c779589b8c27395ced7904efb7de8cacc7 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Milan Zamazal <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
