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

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

 



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
&lt;graphics type='vnc' port='5904'/&gt;

It looks fishy to have one...

&lt;graphics type='rdp' autoport='yes' multiUser='yes' /&gt;
&lt;graphics type='desktop' fullscreen='yes'/&gt;
+&lt;graphics type='vnc'&gt;
+&lt;listen type='address' address='1.2.3.4'/&gt;
+&lt;/graphics&gt;

...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:

&lt;graphics type='vnc' port='5904'&gt;
&lt;listen type='address' address='1.2.3.4'/&gt;
&lt;/graphics&gt;

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!

--
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]