[libvirt] [PATCHv4 1/2] conf: add <listen> subelement to domain <graphics> element

Laine Stump laine at laine.org
Thu Jul 28 17:49:49 UTC 2011


On 07/28/2011 01:09 PM, Eric Blake wrote:
> On 07/28/2011 12:11 AM, Laine Stump wrote:
>> Once it's plugged in, the<listen>  element will be an optional
>> replacement for the "listen" attribute that graphics elements already
>> have. If the<listen>  element is type='address', it will have an
>> attribute called 'address' which will contain an IP address or dns
>> name that the guest's display server should listen on. If, however,
>> type='network', the<listen>  element should have an attribute called
>> 'network' that will be set to the name of a network configuration to
>> get the IP address from.
>
> I think we're there.  Thanks for putting up with the review churn.
>
>> +++ b/docs/formatdomain.html.in
>> @@ -2046,6 +2046,12 @@ qemu-kvm -net nic,model=? /dev/null
>> <graphics type='vnc' port='5904'/>
>
> It looks fishy to have one...
>
>> <graphics type='rdp' autoport='yes' multiUser='yes' />
>> <graphics type='desktop' fullscreen='yes'/>
>> +<graphics type='vnc'>
>> +<listen type='address' address='1.2.3.4'/>
>> +</graphics>
>
> ...and then a second type='vnc' description (do we support multiple 
> vnc graphics adapters on any existing hypervisor?).  How about 
> consolidating this part of the example into just:
>
> <graphics type='vnc' port='5904'>
> <listen type='address' address='1.2.3.4'/>
> </graphics>

Good point. When I added the examples I was only thinking about putting 
in things that would demonstrate the syntax; I didn't think to consider 
if it was actually a practical setup.

>
>> +<dt><code>address</code></dt>
>> +<dd>if<code>type='address'</code>, the<code>address</code>
>> +        attribute will contain either an IP address or hostname (which
>> +        will be resolved to an IP address via a DNS query) to listen
>> +        on. In the "live" XML of a running domain, this attribute be
>
> s/be/will be/
>
> ACK with those nits fixed, no v5 needed.
>

Pushed (!!)  Thanks!




More information about the libvir-list mailing list