Re: [libvirt] [PATCH] Fix serial console telnet protocol support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 14, 2008 at 10:21:25AM +0200, Guido G?nther wrote:
> Hi Mark,
> On Wed, Aug 13, 2008 at 06:44:28PM +0100, Mark McLoughlin wrote:
> > With e.g.:
> > 
> >   <serial type='tcp'>
> >     <source mode='bind' host='127.0.0.1' service='4444'/>
> >     <protocol type='telnet'/>
> >     <target port='0'/>
> >   </serial>
> > 
> > You currently get:
> > 
> >   Unknown option: listen
> >   qemu: could not open serial device 'telnet:127.0.0.1:4444,listen'
> > 
> > With the telnet protocol, qemu expects "server" as an option
> > rather than "listen".
>
> Not that I'm qualified to comment on this but a testcase in
> tests/qemuxml2argv* would prevent breaking this in the future.

The test case is only as good as the person writing the test. We have an
existing test case for Telnet support which is happily validating that we
are generating the incorrect syntax.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]