[Libvirt-ci] Passed: libvirt/libvirt#1634 (v4.7.0-rc2 - fe67e3e)

Travis CI builds at travis-ci.org
Thu Aug 30 08:53:34 UTC 2018


Build Update for libvirt/libvirt
-------------------------------------

Build: #1634
Status: Passed

Duration: 21 mins and 13 secs
Commit: fe67e3e (v4.7.0-rc2)
Author: Luyao Huang
Message: qemu: Validate memory access during validate domain config

Commit 6534b3c4 tried to raise an error when there is no numa
nodes by setting access='shared' in the domain config, but added
a helper called from qemuDomainDeviceDefValidate instead of a
helper called from qemuDomainDefValidate for XML:

  <memoryBacking>
    <hugepages/>
    <access mode='shared'/>
  </memoryBacking>

Since there are no memory devices in the test XML, there would
be no validation failure, but the test added was still failing.
Investigating that it turns out that unnecessary XML elements
were causing the failure (no need for <video>, <graphics>,
<pm>, usb controller model "piix3-uhci", disk attribute for
"discard='unmap'", <serial>, <console>, <channel> and a
memballoon model). Removing all those before moving the method
caused the test to succeed.

So this patch moves the validation to the right place and
removes all the unnecessary XML pieces that were causing
a false validation failure.

https://bugzilla.redhat.com/show_bug.cgi?id=1448149#c14

Signed-off-by: Luyao Huang <lhuang at redhat.com>
Reviewed-by: John Ferlan <jferlan at redhat.com>

View the changeset: https://github.com/libvirt/libvirt/compare/v4.7.0-rc2

View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/422442480?utm_source=email&utm_medium=notification

--

You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications







This email was sent to libvirt-ci at redhat.com (mailto:libvirt-ci at redhat.com)
unsubscribe from this list (http://clicks.travis-ci.com/track/unsub.php?u=14313403&id=ec823542882844fcbbcb258355fa7faa.q3yTAF39gfjNcLjHva7iXjXSQZI%3D&r=https%3A%2F%2Fmandrillapp.com%2Funsub%3Fmd_email%3Dlibvirt-ci%2540redhat.com)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-ci/attachments/20180830/66fe6609/attachment.htm>


More information about the Libvirt-ci mailing list