Change in vdsm[master]: Remove redundant domStatuses parameter from validateAttached...

danken at redhat.com danken at redhat.com
Thu Apr 25 14:42:42 UTC 2013


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 <ewarszaw at redhat.com>
---
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 <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Paikov <paikov at gmail.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list