Francesco Romani has submitted this change and it was merged. Change subject: hsm: lock pool when running upgradeStoragePool ......................................................................
hsm: lock pool when running upgradeStoragePool When running upgradeStoragePool we use the pool metadata to know which domains needs upgrade, having no pool lock means that races with flows that manipulate the pool metadata like activateSd and deactivateSd might occur. Change-Id: I1d5b65a75b1b50d5f5991334cf6221c067a31f5b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1260429 Signed-off-by: Liron Aravot <[email protected]> Reviewed-on: https://gerrit.ovirt.org/45967 Continuous-Integration: Jenkins CI Reviewed-by: Allon Mureinik <[email protected]> Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Nir Soffer <[email protected]> --- M vdsm/storage/hsm.py 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Francesco Romani: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/45967 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d5b65a75b1b50d5f5991334cf6221c067a31f5b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Liron Aravot <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
