Change in vdsm[master]: Simplify SDF.produce() for removing sdf.py.

danken at redhat.com danken at redhat.com
Sat Oct 22 17:56:10 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Simplify SDF.produce() for removing sdf.py.
......................................................................


Simplify SDF.produce() for removing sdf.py.

produce() will return a domain or raise StorageDomainDoesNotExist.
No need for this check.

Change-Id: I442c368a1266855b324eab96e4a7b02634984834
---
M vdsm/storage/sdc.py
M vdsm/storage/sdf.py
2 files changed, 2 insertions(+), 5 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


--
To view, visit http://gerrit.usersys.redhat.com/643
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I442c368a1266855b324eab96e4a7b02634984834
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Yotam Oron <yoron at redhat.com>


More information about the vdsm-patches mailing list