Change in vdsm[master]: exception: Gather exceptions in vdsm.exception

danken at redhat.com danken at redhat.com
Mon Jan 18 16:10:55 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: exception: Gather exceptions in vdsm.exception
......................................................................


exception: Gather exceptions in vdsm.exception

Move GeneralException and ActionStopped from vdsm.utils to
vdsm.exception.

The long term goal is to have one exception module similar to
storage_exception, simplifying error handling and improving the
information returned to clients on errors.

Change-Id: I076a7a381003053e7655ef5bece17c00ce9a92ca
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/48868
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Reviewed-by: Francesco Romani <fromani at redhat.com>
Continuous-Integration: Jenkins CI
---
M lib/vdsm/commands.py
M lib/vdsm/exception.py
M lib/vdsm/qemuimg.py
M lib/vdsm/utils.py
M tests/jobsTests.py
M tests/main.py
M tests/miscTests.py
M vdsm/storage/blockSD.py
M vdsm/storage/blockVolume.py
M vdsm/storage/fileVolume.py
M vdsm/storage/image.py
M vdsm/storage/storage_exception.py
12 files changed, 51 insertions(+), 47 deletions(-)

Approvals:
  Piotr Kliczewski: Looks good to me, approved
  Nir Soffer: Verified
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/48868
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I076a7a381003053e7655ef5bece17c00ce9a92ca
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list