Change in ovirt-hosted-engine-ha[master]: Load 'virtio-scsi' controller from OVF as 'scsi' controller.

Code Review gerrit at ovirt.org
Fri Aug 25 12:32:48 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged. ( https://gerrit.ovirt.org/80854 )

Change subject: Load 'virtio-scsi' controller from OVF as 'scsi' controller.
......................................................................


Load 'virtio-scsi' controller from OVF as 'scsi' controller.

The engine stores 'virtio-scsi' controller to OVF, but vdsm and
libvirt recognize only 'scsi' controller.
The name is changed when loading from OVF.

Change-Id: I9cd9ac11c77ac34bd5c647d0661289263bff74c0
Bug-Url: https://bugzilla.redhat.com/1479776
Signed-off-by: Andrej Krejcir <akrejcir at redhat.com>
---
M ovirt_hosted_engine_ha/lib/ovf/ovf2VmParams.py
M ovirt_hosted_engine_ha/lib/ovf/ovf_test.xml
M ovirt_hosted_engine_ha/lib/ovf/ovf_test_max_vcpu.xml
3 files changed, 5 insertions(+), 2 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Andrej Krejcir: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9cd9ac11c77ac34bd5c647d0661289263bff74c0
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list