Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm: use vdscli instead of vdsClient ...................................................................... vdsm: use vdscli instead of vdsClient Dropping vdsClient usage in favor of vdscli API. Using vdscli allows to achieve a better error handling and to have more control on what's going on. Bug-Url: https://bugzilla.redhat.com/1144334 Bug-Url: https://bugzilla.redhat.com/1101553 Bug-Url: https://bugzilla.redhat.com/1142098 Related-To: https://bugzilla.redhat.com/1005923 Related-To: https://bugzilla.redhat.com/1150427 Change-Id: I0f17abcc4ec83a69832bdb5a986136831504da2c Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com&gt; Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; --- M src/ovirt_hosted_engine_setup/constants.py M src/ovirt_hosted_engine_setup/mixins.py M src/ovirt_hosted_engine_setup/tasks.py M src/plugins/ovirt-hosted-engine-setup/engine/add_disk.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py M src/plugins/ovirt-hosted-engine-setup/engine/os_install.py M src/plugins/ovirt-hosted-engine-setup/ha/ha_services.py M src/plugins/ovirt-hosted-engine-setup/network/bridge.py M src/plugins/ovirt-hosted-engine-setup/storage/nfs.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py M src/plugins/ovirt-hosted-engine-setup/system/vdsmenv.py M src/plugins/ovirt-hosted-engine-setup/vdsmd/cpu.py M src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py M src/plugins/ovirt-hosted-engine-setup/vm/configurevm.py M src/plugins/ovirt-hosted-engine-setup/vm/image.py M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py 16 files changed, 264 insertions(+), 148 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f17abcc4ec83a69832bdb5a986136831504da2c Gerrit-PatchSet: 18 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Antoni Segura Puimedon <toni+ovirt(a)midokura.com&gt; Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt; Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt; Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com&gt; Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com&gt; Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com&gt; Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com&gt; Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server