Dan Kenigsberg has submitted this change and it was merged. Change subject: virt: move handling of recovery file in a class ......................................................................
virt: move handling of recovery file in a class saveState() is little more than a fancy wrapper around pickle, tailored for the Vm special case. Extract all the saveState() related code in a small helper class, and move it in to recovery.py, to make the code easier to read and test. Change-Id: I6866eb2c3c9d5784c2054c860cbf0977e8e58031 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/51388 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/recovery.py M vdsm/virt/vm.py 2 files changed, 76 insertions(+), 44 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/51388 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6866eb2c3c9d5784c2054c860cbf0977e8e58031 Gerrit-PatchSet: 12 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: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
