Bug 1285665 - Fail to valid the guest's xml while set the graphical listen as ipv6 address which end with "::" on rhel7
Fail to valid the guest's xml while set the graphical listen as ipv6 address ...
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: libvirt (Show other bugs)
x86_64 Unspecified
medium Severity medium
: rc
: ---
Assigned To: Ján Tomko
Virtualization Bugs
Depends On:
  Show dependency treegraph
Reported: 2015-11-26 03:27 EST by mxie@redhat.com
Modified: 2016-11-03 14:32 EDT (History)
9 users (show)

See Also:
Fixed In Version: libvirt-1.3.1-1.el7
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2016-11-03 14:32:18 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description mxie@redhat.com 2015-11-26 03:27:39 EST
Description of problem:
Fail to valid the guest's xml while set the graphical listen as ipv6 address which end with "::" on rhel7

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1.Edit a guest xml on rhel7 and change graphical listen address to "::" or other ipv6 address end with "::"
# virsh edit 2OS
<graphics type='vnc' port='5901' autoport='no' listen='::'>
  <listen type='address' address='::'/>

2.Using ":wq" to save xml, it will pop up below error
# virsh edit 2OS
error: XML document failed to validate against schema: Unable to validate doc against /usr/share/libvirt/schemas/domain.rng
Extra element devices in interleave
Element domain failed to validate content

Failed. Try again? [y,n,i,f,?]: 

3.if type "i" , the xml could be saved as below and the graphical listen ipv6 address "::" could be used by guest normally

Domain 2OS XML configuration edited.

Actual results:
As above description

Expected results:
Could valid the guest's xml while set the graphical listen as ipv6 address which end with "::" on rhel7 

Addtional info:
Could valid the guest's xml while set the graphical listen as ipv6 address which end with "::" on rhel6 normally
Comment 1 Ján Tomko 2015-11-26 04:52:37 EST
Reproducible with libvirt-1.2.17-13.el7

Upstream patch:
Comment 2 Ján Tomko 2015-11-26 04:56:38 EST
Note that 'virsh edit' did not validate the XML before RHEL 7.2, it was added upstream by:

commit d7565bd8aab018302307a994e7753766cc72c085
Author:     Daniel P. Berrange <berrange@redhat.com>
CommitDate: 2015-01-16 11:07:59 +0000

    virsh: add support for domain XML validation
    The 'virsh edit' command gets XML validation enabled by default,
    with a --skip-validate option to disable it. The 'virsh define'
    and 'virsh create' commands get a --validate option to enable
    it, to avoid regressions for existing scripts.
    The quality of error reporting from libxml2 varies depending
    on the type of XML error made. Sometimes it is quite clear
    and useful, other times it is obscure & inaccurate. At least
    the user will see an error now, rather than having their
    XML modification silently disappear.

git describe: v1.2.11-188-gd7565bd contains: v1.2.12-rc1~21
Comment 3 Ján Tomko 2015-11-26 05:06:06 EST
Now pushed upstream:
commit 7ec235ed092a2408ca0d3b7fd1c4bfcbc3fc835e
Author:     Ján Tomko <jtomko@redhat.com>
CommitDate: 2015-11-26 11:05:18 +0100

    schema: use a better regex for listen addresses
    A domain with '::' as the listen address fails to validate.
    Reuse the 'ipAddr' and 'dnsName' regexes from basictypes instead
    of reinventing them.

git describe: v1.2.21-142-g7ec235e
Comment 5 Fangge Jin 2016-03-02 21:14:54 EST
Verify pass on downstream build: libvirt-1.3.1-1.el7.x86_64

# virsh edit rhel7
(edit listen address to :: or 0::, like
<graphics type='spice' port='5900' autoport='no' listen='0::'>
Domain rhel7 XML configuration edited.

# virsh dumpxml rhel7 |grep gra
<graphics type='spice' port='5900' autoport='no' listen='0::'>

# virsh start rhel7

# ps aux|grep qemu
...-spice port=5900,addr=::,disable-ticketing,seamless-migration=on...
Comment 6 Fangge Jin 2016-04-10 23:53:58 EDT
Verify pass on downstream build: libvirt-1.3.3-1.el7.x86_64

1. # virsh edit windows
    <graphics type='spice' port='5905' autoport='no' listen='**0::**'/>
Domain windows XML configuration edited.
2. # virsh dumpxml windows
    <graphics type='spice' port='5905' autoport='no' listen='0::'>
      <listen type='address' address='0::'/>
3. # virsh start windows
4. # ps aux|grep qemu
-spice port=5905,addr=0::,...
5.# remote-viewer spice://
6.Edit listen address to '::', and test again
Comment 8 errata-xmlrpc 2016-11-03 14:32:18 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.