Dan Kenigsberg has submitted this change and it was merged.

Change subject: lib: move exception.py under common
......................................................................


lib: move exception.py under common

The commit fc732ae broke the rule that code under lib/vdsm/common
should depend only on code under the same subtree.

The good news is that the code moved in fc732ae depends only on
a few modules; one of them is exception.py, which makes sense
to move under common/ anyway.

This patch does that.

Change-Id: I7742aa7dafe0e0e29044536ad19fbddbcab0f420
Signed-off-by: Francesco Romani <from...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/63952
Continuous-Integration: Jenkins CI
Reviewed-by: Nir Soffer <nsof...@redhat.com>
Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com>
---
M lib/vdsm/Makefile.am
M lib/vdsm/commands.py
M lib/vdsm/common/api.py
R lib/vdsm/common/exception.py
M lib/vdsm/define.py
M lib/vdsm/gluster/exception.py
M lib/vdsm/hooks.py
M lib/vdsm/jobs.py
M lib/vdsm/qemuimg.py
M lib/vdsm/rpc/Bridge.py
M lib/vdsm/rpc/bindingxmlrpc.py
M lib/vdsm/storage/exception.py
M lib/yajsonrpc/__init__.py
M tests/api_test.py
M tests/bridgeTests.py
M tests/exception_test.py
M tests/jobsTests.py
M tests/miscTests.py
M tests/qemuimgTests.py
M tests/storage_exception_test.py
M tests/storage_sdm_create_volume_test.py
M vdsm.spec.in
M vdsm/API.py
M vdsm/storage/blockSD.py
M vdsm/storage/blockVolume.py
M vdsm/storage/fileVolume.py
M vdsm/storage/image.py
M vdsm/storage/sdm/api/create_volume.py
28 files changed, 28 insertions(+), 27 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7742aa7dafe0e0e29044536ad19fbddbcab0f420
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to