Vinzenz Feenstra has posted comments on this change.

Change subject: vm.py: Always use 'qemu' domain type in fake mode
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

http://gerrit.ovirt.org/#/c/23938/1/vdsm/vm.py
File vdsm/vm.py:

Line 914:         self.arch = arch
Line 915: 
Line 916:         self.doc = xml.dom.minidom.Document()
Line 917: 
Line 918:         if not config.getboolean('vars', 'fake_kvm_support') and \
This is exactly what I thought what was the problem
Line 919:                 utils.tobool(self.conf.get('kvmEnable', 'true')):
Line 920:             domainType = 'kvm'
Line 921:         else:
Line 922:             domainType = 'qemu'


-- 
To view, visit http://gerrit.ovirt.org/23938
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I875854ccc4d884aa36a77cd062521889622e567d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vitor de Lima <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to