Dan Kenigsberg has submitted this change and it was merged.

Change subject: lib: move the response.py under common
......................................................................


lib: move the response.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 last module we need to move to fix the breakage is 'response'.
Once again, this makes sense to be moved in the common layer anyway.

Change-Id: I6263fb8991a226516ba5d14e909a4a70ea0078d2
Signed-off-by: Francesco Romani <from...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/63954
Continuous-Integration: Jenkins CI
Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com>
---
M lib/vdsm/Makefile.am
M lib/vdsm/common/api.py
R lib/vdsm/common/response.py
M lib/vdsm/jobs.py
M lib/vdsm/jsonrpcvdscli.py
M lib/vdsm/v2v.py
M lib/vdsm/virt/secret.py
M lib/vdsm/virt/vmpowerdown.py
M tests/api_test.py
M tests/jobsTests.py
M tests/responseTests.py
M tests/vmMigrationTests.py
M tests/vmOperationsTests.py
M tests/vmRecoveryTests.py
M tests/vmSecretTests.py
M tests/vmTests.py
M tests/vmfakelib.py
M vdsm.spec.in
M vdsm/API.py
M vdsm/clientIF.py
M vdsm/virt/migration.py
M vdsm/virt/recovery.py
M vdsm/virt/vm.py
23 files changed, 22 insertions(+), 25 deletions(-)

Approvals:
  Piotr Kliczewski: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6263fb8991a226516ba5d14e909a4a70ea0078d2
Gerrit-PatchSet: 4
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