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

Code Review gerrit at ovirt.org
Wed Jan 4 08:32:30 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:
  Piotr Kliczewski: Looks good to me, but someone else must approve
  Simone Tiraboschi: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4880eb94dfabe9628d34687a97febe3b1c6fd969
Gerrit-PatchSet: 26
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Rosenzwig <irosenzw at redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list