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

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(a)redhat.com&gt; Reviewed-by: John Ferlan <jferlan(a)redhat.com&gt; 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&a... -- 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(a)redhat.com (mailto:libvirt-ci@redhat.com) unsubscribe from this list (http://clicks.travis-ci.com/track/unsub.php?u=14313403&id=ec823542882...)