Change in vdsm[master]: virt: Clarify error handling
danken at redhat.com
danken at redhat.com
Thu Feb 26 17:46:44 UTC 2015
Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt: Clarify error handling
......................................................................
virt: Clarify error handling
In diskSizeExtend, error handling was correct but less clear then it
should be. We have try except block, returning one value on success, and
other value on errors.
Change-Id: I1d8f10e3ac46f831ddaea8a30ba289370c697b0e
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/37944
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Adam Litke <alitke at redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
Adam Litke: Looks good to me, approved
Nir Soffer: Verified
Francesco Romani: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/37944
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1d8f10e3ac46f831ddaea8a30ba289370c697b0e
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Candace Sheremeta <cshereme at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list