Dan Kenigsberg has submitted this change and it was merged. Change subject: Remove redundant domStatuses parameter from validateAttachedDomain() ......................................................................
Remove redundant domStatuses parameter from validateAttachedDomain() The only possible status of a SD belonging to a pool are active or attached. Then the domain status is irrelevant when checking membership to the pool. Change-Id: Ib3daea2fa5928dee18efdf6452b65fbc212eec54 Signed-off-by: Eduardo <[email protected]> --- M vdsm/storage/sp.py 1 file changed, 4 insertions(+), 5 deletions(-) Approvals: Yeela Kaplan: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Vered Volansky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14012 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib3daea2fa5928dee18efdf6452b65fbc212eec54 Gerrit-PatchSet: 2 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: Daniel Paikov <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Vered Volansky <[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
