Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.1]: jsonrpc: python jsonrpcvdscli instead of vdsClient

Code Review gerrit at ovirt.org
Wed Jan 4 12:44:25 UTC 2017


>From Simone Tiraboschi <stirabos at redhat.com>:

Simone Tiraboschi has submitted this change and it was merged.

Change subject: jsonrpc: python jsonrpcvdscli instead of vdsClient
......................................................................


jsonrpc: python jsonrpcvdscli instead of vdsClient

Use python jsonrpcvdscli instead of vdsClient for
ancillary commands on bash scripts.
Adding unit test for the vm.conf parsing code based
on pytest.
Adding the initial pytest configuration.

Change-Id: I4880eb94dfabe9628d34687a97febe3b1c6fd969
Bug-Url: https://bugzilla.redhat.com/1393918
Signed-off-by: Simone Tiraboschi <stirabos at redhat.com>
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M .gitignore
M Makefile.am
M ovirt-hosted-engine-setup.spec.in
M po/POTFILES.in
A pytest.ini
M src/bin/hosted-engine.in
M src/ovirt_hosted_engine_setup/Makefile.am
A src/ovirt_hosted_engine_setup/vdsm_helper.py
A src/ovirt_hosted_engine_setup/vm_test.conf
A src/ovirt_hosted_engine_setup/vmconf.py
A src/ovirt_hosted_engine_setup/vmconf_test.py
M src/plugins/gr-he-upgradeappliance/vm/runvm.py
12 files changed, 534 insertions(+), 55 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Simone Tiraboschi: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests
  Rafael Martins: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4880eb94dfabe9628d34687a97febe3b1c6fd969
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: ovirt-hosted-engine-setup-2.1
Gerrit-Owner: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Rafael Martins <rmartins at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list