Dan Kenigsberg has submitted this change and it was merged. Change subject: Move hsm.validatePoolSD() -> pool.validatePoolSD(). ......................................................................
Move hsm.validatePoolSD() -> pool.validatePoolSD(). Since StorageDomainNotMemberOfPool and StorageDomainNotInPool are handled the same in the Engine is better to raise the proper Exception in spite that before StoragePool.validateAttachedDomain() raised the 2nd one only. Related to BZ#878894. Change-Id: I3ba16c8a427c9478ae81c2693d79b8bd601f1ec3 Signed-off-by: Eduardo <[email protected]> --- M vdsm/storage/hsm.py M vdsm/storage/sp.py 2 files changed, 23 insertions(+), 28 deletions(-) Approvals: Dan Kenigsberg: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9455 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ba16c8a427c9478ae81c2693d79b8bd601f1ec3 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Eduardo <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Royce Lv <[email protected]> Gerrit-Reviewer: Shu Ming <[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
