Francesco Romani has submitted this change and it was merged. Change subject: sp: deactivateSd - remove domain from pending for upgrade list ......................................................................
sp: deactivateSd - remove domain from pending for upgrade list When domain is deactivated its not cleared from the _domainsToUpgrade list, which causes it to hold wrong and unneeded information (the domain might be detached later from the storage pool). This patch makes sure the domain is cleared when deactivated. Change-Id: I4451b348b8837dd83d95aea2be4a4536b33cdd99 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1260429 Signed-off-by: Liron Aravot <[email protected]> Reviewed-on: https://gerrit.ovirt.org/45968 Continuous-Integration: Jenkins CI Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Nir Soffer <[email protected]> --- M vdsm/storage/sp.py 1 file changed, 17 insertions(+), 8 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/45968 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4451b348b8837dd83d95aea2be4a4536b33cdd99 Gerrit-PatchSet: 10 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
