Dan Kenigsberg has submitted this change and it was merged. Change subject: sp: improve masterMigrate safety ......................................................................
sp: improve masterMigrate safety The goal of this patch is to clean, optimize and give robustness to the masterMigrate method. In this patch: - acquire the cluster lock before start using the new master filesystem - refactor out all the required metadata changes in switchMasterDomain - divide the method in two parts (operations required to switch to the new master and previous master cleanup) - address the two parts exception handling (hard failure for the first, logging only for the second) Change-Id: I155b36d41df43230f99ba610699bba05bee3f0c0 Signed-off-by: Federico Simoncelli <[email protected]> Reviewed-on: http://gerrit.ovirt.org/22419 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Ayal Baron <[email protected]> --- M vdsm/storage/sp.py 1 file changed, 83 insertions(+), 94 deletions(-) Approvals: Ayal Baron: Looks good to me, approved Nir Soffer: Looks good to me, but someone else must approve Federico Simoncelli: Verified -- To view, visit http://gerrit.ovirt.org/22419 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I155b36d41df43230f99ba610699bba05bee3f0c0 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
