Change in vdsm[master]: vm: Automatically add a serial port for a console device
vfeenstr at redhat.com
vfeenstr at redhat.com
Mon Mar 24 08:15:15 UTC 2014
Vinzenz Feenstra has posted comments on this change.
Change subject: vm: Automatically add a serial port for a console device
......................................................................
Patch Set 4:
(1 comment)
http://gerrit.ovirt.org/#/c/25979/4/vdsm/vm.py
File vdsm/vm.py:
Line 1170: commandLine.appendChildWithArgs('qemu:arg', value='-usbdevice')
Line 1171: commandLine.appendChildWithArgs('qemu:arg', value='keyboard')
Line 1172: self.dom.appendChild(commandLine)
Line 1173:
Line 1174: def appendSerial(self):
> why not part of ConsoleDevice's getXML?
I tried to, but it will fail as it expects only one element :(
Line 1175: """
Line 1176: Add a serial port for the console device if it exists.
Line 1177:
Line 1178: <serial type='pty'>
--
To view, visit http://gerrit.ovirt.org/25979
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa7b02a7bcaad63017c35c811a194fa42e2b694f
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list