Dan Kenigsberg has submitted this change and it was merged. Change subject: lib: move define.py under common ......................................................................
lib: move define.py under common The commit fc732ae broke the rule that code under lib/vdsm/common should depend only on code under the same subtree. Another module which should be moved to fix the breakage is define.py. Again this makes sense to move anyway. It should go away soon, but for the meantime let's have this here. Change-Id: I634d3bac718fe0615a2aca951ec47b976967ada1 Signed-off-by: Francesco Romani <from...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/63953 Continuous-Integration: Jenkins CI Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> --- M lib/vdsm/Makefile.am R lib/vdsm/common/define.py M lib/vdsm/host/api.py M lib/vdsm/momIF.py M lib/vdsm/response.py M lib/vdsm/rpc/bindingxmlrpc.py M lib/vdsm/v2v.py M lib/vdsm/virt/vmpowerdown.py M tests/functional/momTests.py M tests/momTests.py M tests/responseTests.py M tests/vmApiTests.py M tests/vmOperationsTests.py M tests/vmTests.py M vdsm.spec.in M vdsm/API.py M vdsm/clientIF.py M vdsm/gluster/api.py M vdsm/virt/migration.py M vdsm/virt/vm.py 20 files changed, 19 insertions(+), 21 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/63953 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I634d3bac718fe0615a2aca951ec47b976967ada1 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
