- CI - libvirt List Archives

[Libvirt-ci] Errored: libvirt/libvirt#1726 (master - 0615c84)
by Travis CI 27 Sep '18

27 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1726 Status: Errored Duration: 19 mins and 33 secs Commit: 0615c84 (master) Author: Pavel Hrdina Message: vircgroupv1: fix build on non-linux OSes Cgroups are linux specific and we need to make sure that the code is compiled only on linux. On different OSes it fails the compilation: ../../src/util/vircgroupv1.c:65:19: error: variable has incomplete type 'struct mntent' struct mntent entry; ^ ../../src/util/vircgroupv1.c:65:12: note: forward declaration of 'struct mntent' struct mntent entry; ^ ../../src/util/vircgroupv1.c:74:12: error: implicit declaration of function 'getmntent_r' is invalid in C99 [-Werror,-Wimplicit-function-declaration] while (getmntent_r(mounts, &entry, buf, sizeof(buf)) != NULL) { ^ ../../src/util/vircgroupv1.c:814:39: error: use of undeclared identifier 'MS_NOSUID' if (mount("tmpfs", root, "tmpfs", MS_NOSUID|MS_NODEV|MS_NOEXEC, opts) < 0) { ^ ../../src/util/vircgroupv1.c:814:49: error: use of undeclared identifier 'MS_NODEV' if (mount("tmpfs", root, "tmpfs", MS_NOSUID|MS_NODEV|MS_NOEXEC, opts) < 0) { ^ ../../src/util/vircgroupv1.c:814:58: error: use of undeclared identifier 'MS_NOEXEC' if (mount("tmpfs", root, "tmpfs", MS_NOSUID|MS_NODEV|MS_NOEXEC, opts) < 0) { ^ ../../src/util/vircgroupv1.c:841:65: error: use of undeclared identifier 'MS_BIND' if (mount(src, group->legacy[i].mountPoint, "none", MS_BIND, ^ Reviewed-by: Ján Tomko <jtomko(a)redhat.com> Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/b7ccd0757de7...0615c8436ac8 View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/434038817?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Build failed in Jenkins: libvirt+mingw32-build » libvirt-fedora-rawhide #64
by ci＠centos.org 27 Sep '18

27 Sep '18

See <https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…> Changes: [phrdina] util: introduce vircgroupbackend files [phrdina] vircgroup: introduce cgroup v1 backend files [phrdina] vircgroup: extract virCgroupV1Available [phrdina] vircgroup: detect available backend for cgroup [phrdina] vircgroup: extract virCgroupV1ValidateMachineGroup [phrdina] vircgroup: extract virCgroupV1CopyMounts [phrdina] vircgroup: extract v1 detect functions [phrdina] vircgroup: extract virCgroupV1CopyPlacement [phrdina] vircgroup: extract virCgroupV1ValidatePlacement [phrdina] vircgroup: extract virCgroupV1StealPlacement [phrdina] vircgroup: extract virCgroupV1DetectControllers [phrdina] vircgroup: extract virCgroupV1HasController [phrdina] vircgroup: extract virCgroupV1GetAnyController [phrdina] vircgroup: extract virCgroupV1PathOfController [phrdina] vircgroup: extract virCgroupV1MakeGroup [phrdina] vircgroup: extract virCgroupV1Remove [phrdina] vircgroup: extract virCgroupV1AddTask [phrdina] vircgroup: extract virCgroupV1HasEmptyTasks [phrdina] vircgroup: extract virCgroupV1BindMount [phrdina] vircgroup: extract virCgroupV1SetOwner [phrdina] vircgroup: extract virCgroupV1(Set|Get)BlkioWeight [phrdina] vircgroup: extract virCgroupV1GetBlkioIoServiced [phrdina] vircgroup: extract virCgroupV1GetBlkioIoDeviceServiced [phrdina] vircgroup: extract virCgroupV1(Set|Get)BlkioDeviceWeight [phrdina] vircgroup: extract virCgroupV1(Set|Get)BlkioDeviceReadIops [phrdina] vircgroup: extract virCgroupV1(Set|Get)BlkioDeviceWriteIops [phrdina] vircgroup: extract virCgroupV1(Set|Get)BlkioDeviceReadBps [phrdina] vircgroup: extract virCgroupV1(Set|Get)BlkioDeviceWriteBps [phrdina] vircgroup: extract virCgroupV1SetMemory [phrdina] vircgroup: extract virCgroupV1GetMemoryStat [phrdina] vircgroup: extract virCgroupV1GetMemoryUsage [phrdina] vircgroup: extract virCgroupV1(Set|Get)Memory*Limit [phrdina] vircgroup: extract virCgroupV1GetMemSwapUsage [phrdina] vircgroup: extract virCgroupV1(Allow|Deny)Device [phrdina] vircgroup: extract virCgroupV1(Allow|Deny)AllDevices [phrdina] vircgroup: extract virCgroupV1(Set|Get)CpuShares [phrdina] vircgroup: extract virCgroupV1(Set|Get)CpuCfsPeriod [phrdina] vircgroup: extract virCgroupV1(Set|Get)CpuCfsQuota [phrdina] vircgroup: extract virCgroupV1SupportsCpuBW [phrdina] vircgroup: extract virCgroupV1GetCpuacct*Usage [phrdina] vircgroup: extract virCgroupV1GetCpuacctStat [phrdina] vircgroup: extract virCgroupV1(Set|Get)FreezerState [phrdina] vircgroup: extract virCgroupV1(Set|Get)CpusetMems [phrdina] vircgroup: extract virCgroupV1(Set|Get)CpusetMemoryMigrate [phrdina] vircgroup: extract virCgroupV1(Set|Get)CpusetCpus [phrdina] vircgroup: rename virCgroupController into virCgroupV1Controller [phrdina] vircgroup: rename controllers to legacy ------------------------------------------ [...truncated 129.01 KB...] CC perror.lo CC pipe.lo CC poll.lo CC posix_openpt.lo CC printf-args.lo CC printf-parse.lo CC pthread_sigmask.lo CC raise.lo CC rawmemchr.lo CC readlink.lo CC realloc.lo CC recv.lo CC regex.lo CC secure_getenv.lo CC select.lo CC send.lo CC setenv.lo CC setsockopt.lo CC sigaction.lo CC sigprocmask.lo CC sleep.lo CC snprintf.lo CC socket.lo CC stat.lo CC stat-w32.lo CC stdio-read.lo CC stdio-write.lo CC stpcpy.lo CC strchrnul.lo CC strdup.lo CC strerror.lo CC strerror-override.lo CC strerror_r.lo CC strndup.lo CC strptime.lo CC strsep.lo CC time_r.lo CC timegm.lo CC ttyname_r.lo CC uname.lo CC unsetenv.lo CC usleep.lo CC vasnprintf.lo CC vsnprintf.lo CC waitpid.lo CC wcwidth.lo CC glthread/lock.lo CC uniwidth/width.lo CCLD libgnu.la gmake[3]: Leaving directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> gmake[2]: Leaving directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> Making all in include/libvirt gmake[2]: Entering directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> gmake[2]: Nothing to be done for 'all'. gmake[2]: Leaving directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> Making all in src gmake[2]: Entering directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> GEN util/virkeycodetable_linux.h GEN util/virkeycodetable_osx.h GEN util/virkeycodetable_atset1.h GEN util/virkeycodetable_atset2.h GEN util/virkeycodetable_atset3.h GEN util/virkeycodetable_xtkbd.h GEN util/virkeycodetable_usb.h GEN util/virkeycodetable_win32.h GEN util/virkeycodetable_qnum.h GEN util/virkeynametable_linux.h GEN util/virkeynametable_osx.h GEN util/virkeynametable_win32.h GEN ../../src/access/org.libvirt.api.policy GEN access/viraccessapicheck.h GEN access/viraccessapicheck.c GEN access/viraccessapicheckqemu.h GEN access/viraccessapicheckqemu.c GEN access/viraccessapichecklxc.h GEN access/viraccessapichecklxc.c GEN libvirt_access.xml GEN libvirt_access_qemu.xml GEN libvirt_access_lxc.xml GEN logging/log_protocol.h GEN logging/log_daemon_dispatch_stubs.h GEN locking/lock_protocol.h GEN locking/lock_daemon_dispatch_stubs.h GEN admin/admin_protocol.h GEN admin/admin_client.h GEN admin/admin_server_dispatch_stubs.h GEN rpc/virnetprotocol.h GEN rpc/virkeepaliveprotocol.h GEN .esx_vi_generator.stamp GEN .hyperv_wmi_generator.stamp GEN lxc/lxc_monitor_protocol.h GEN lxc/lxc_monitor_dispatch.h GEN ../../src/hyperv/hyperv_wmi_classes.generated.typedef GEN ../../src/hyperv/hyperv_wmi_classes.generated.h GEN ../../src/hyperv/hyperv_wmi_classes.generated.c GEN lxc/lxc_controller_dispatch.h GEN ../../src/esx/esx_vi_types.generated.typedef GEN ../../src/esx/esx_vi_types.generated.typeenum GEN ../../src/esx/esx_vi_types.generated.typetostring GEN ../../src/esx/esx_vi_types.generated.typefromstring GEN ../../src/esx/esx_vi_types.generated.h GEN ../../src/esx/esx_vi_types.generated.c GEN ../../src/esx/esx_vi_methods.generated.h GEN ../../src/esx/esx_vi_methods.generated.c GEN ../../src/esx/esx_vi_methods.generated.macro GEN ../../src/esx/esx_vi.generated.h GEN ../../src/esx/esx_vi.generated.c GEN remote/remote_protocol.h GEN remote/remote_client_bodies.h GEN remote/lxc_protocol.h GEN remote/lxc_client_bodies.h GEN remote/qemu_protocol.h GEN remote/qemu_client_bodies.h GEN remote/remote_daemon_dispatch_stubs.h GEN remote/remote_daemon_dispatch_lxc_stubs.h GEN remote/remote_daemon_dispatch_qemu_stubs.h GEN libvirt_access.syms GEN libvirt_access_qemu.syms GEN libvirt_access_lxc.syms GEN libvirt_qemu.def GEN libvirt_lxc.def GEN libvirt_admin.syms GEN libvirtd.qemu.logrotate GEN libvirtd.lxc.logrotate GEN libvirtd.libxl.logrotate GEN libvirtd.uml.logrotate GEN libvirtd.logrotate GEN logging/log_protocol.c GEN locking/lock_protocol.c GEN admin/admin_protocol.c GEN rpc/virnetprotocol.c GEN rpc/virkeepaliveprotocol.c GEN lxc/lxc_monitor_protocol.c GEN remote/remote_protocol.c GEN remote/lxc_protocol.c GEN remote/qemu_protocol.c GEN libvirt.syms GEN libvirt_admin.def GEN libvirt.def /usr/bin/gmake all-am gmake[3]: Entering directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> CC libvirt_driver_la-driver.lo CC libvirt_driver_la-datatypes.lo CC libvirt_driver_la-libvirt.lo CC libvirt_driver_la-libvirt-domain.lo CC libvirt_driver_la-libvirt-domain-snapshot.lo CC libvirt_driver_la-libvirt-host.lo CC libvirt_driver_la-libvirt-interface.lo CC libvirt_driver_la-libvirt-network.lo CC libvirt_driver_la-libvirt-nodedev.lo CC libvirt_driver_la-libvirt-nwfilter.lo CC libvirt_driver_la-libvirt-secret.lo CC libvirt_driver_la-libvirt-storage.lo CC libvirt_driver_la-libvirt-stream.lo CC libvirt_qemu_la-libvirt-qemu.lo CC libvirt_lxc_la-libvirt-lxc.lo CC libvirt_admin_la-libvirt-admin.lo CC libvirt_admin_la-datatypes.lo CC admin/libvirt_driver_admin_la-admin_protocol.lo CC admin/libvirt_driver_admin_la-admin_server.lo CC admin/libvirt_driver_admin_la-admin_server_dispatch.lo GEN util/virkeycode-linux.pod GEN util/virkeycode-osx.pod GEN util/virkeycode-atset1.pod GEN util/virkeycode-atset2.pod GEN util/virkeycode-atset3.pod GEN util/virkeycode-xtkbd.pod GEN util/virkeycode-usb.pod GEN util/virkeycode-win32.pod GEN util/virkeycode-qnum.pod GEN util/virkeyname-linux.pod GEN util/virkeyname-osx.pod GEN util/virkeyname-win32.pod CC util/libvirt_util_la-viralloc.lo CC util/libvirt_util_la-virarch.lo CC util/libvirt_util_la-virarptable.lo CC util/libvirt_util_la-viratomic.lo CC util/libvirt_util_la-viraudit.lo CC util/libvirt_util_la-virauth.lo CC util/libvirt_util_la-virauthconfig.lo CC util/libvirt_util_la-virbitmap.lo CC util/libvirt_util_la-virbuffer.lo CC util/libvirt_util_la-virperf.lo CC util/libvirt_util_la-vircgroup.lo CC util/libvirt_util_la-vircgroupbackend.lo CC util/libvirt_util_la-vircgroupv1.lo CC util/libvirt_util_la-virclosecallbacks.lo CC util/libvirt_util_la-vircommand.lo CC util/libvirt_util_la-virconf.lo ../../src/util/vircgroupv1.c: In function 'virCgroupV1Available': ../../src/util/vircgroupv1.c:65:19: error: storage size of 'entry' isn't known struct mntent entry; ^~~~~ ../../src/util/vircgroupv1.c:74:12: error: implicit declaration of function 'getmntent_r'; did you mean 'getEntity'? [-Werror=implicit-function-declaration] while (getmntent_r(mounts, &entry, buf, sizeof(buf)) != NULL) { ^~~~~~~~~~~ getEntity ../../src/util/vircgroupv1.c:74:12: error: nested extern declaration of 'getmntent_r' [-Werror=nested-externs] ../../src/util/vircgroupv1.c:65:19: error: unused variable 'entry' [-Werror=unused-variable] struct mntent entry; ^~~~~ ../../src/util/vircgroupv1.c: In function 'virCgroupV1BindMount': ../../src/util/vircgroupv1.c:814:9: error: implicit declaration of function 'mount' [-Werror=implicit-function-declaration] if (mount("tmpfs", root, "tmpfs", MS_NOSUID|MS_NODEV|MS_NOEXEC, opts) < 0) { ^~~~~ ../../src/util/vircgroupv1.c:814:9: error: nested extern declaration of 'mount' [-Werror=nested-externs] ../../src/util/vircgroupv1.c:814:39: error: 'MS_NOSUID' undeclared (first use in this function); did you mean 'S_ISUID'? if (mount("tmpfs", root, "tmpfs", MS_NOSUID|MS_NODEV|MS_NOEXEC, opts) < 0) { ^~~~~~~~~ S_ISUID ../../src/util/vircgroupv1.c:814:39: note: each undeclared identifier is reported only once for each function it appears in ../../src/util/vircgroupv1.c:814:49: error: 'MS_NODEV' undeclared (first use in this function); did you mean 'ENODEV'? if (mount("tmpfs", root, "tmpfs", MS_NOSUID|MS_NODEV|MS_NOEXEC, opts) < 0) { ^~~~~~~~ ENODEV ../../src/util/vircgroupv1.c:814:58: error: 'MS_NOEXEC' undeclared (first use in this function); did you mean 'S_IEXEC'? if (mount("tmpfs", root, "tmpfs", MS_NOSUID|MS_NODEV|MS_NOEXEC, opts) < 0) { ^~~~~~~~~ S_IEXEC ../../src/util/vircgroupv1.c:841:65: error: 'MS_BIND' undeclared (first use in this function); did you mean 'DC_BINS'? if (mount(src, group->legacy[i].mountPoint, "none", MS_BIND, ^~~~~~~ DC_BINS ../../src/util/vircgroupv1.c:854:17: error: implicit declaration of function 'symlink'; did you mean 'unlink'? [-Werror=implicit-function-declaration] if (symlink(group->legacy[i].mountPoint, ^~~~~~~ unlink ../../src/util/vircgroupv1.c:854:17: error: nested extern declaration of 'symlink' [-Werror=nested-externs] ../../src/util/vircgroupv1.c: In function 'virCgroupV1GetCpuacctStat': ../../src/util/vircgroupv1.c:1916:30: error: implicit declaration of function 'sysconf'; did you mean 'swscanf'? [-Werror=implicit-function-declaration] long ticks_per_sec = sysconf(_SC_CLK_TCK); ^~~~~~~ swscanf ../../src/util/vircgroupv1.c:1916:30: error: nested extern declaration of 'sysconf' [-Werror=nested-externs] ../../src/util/vircgroupv1.c:1916:38: error: '_SC_CLK_TCK' undeclared (first use in this function); did you mean 'CLK_TCK'? long ticks_per_sec = sysconf(_SC_CLK_TCK); ^~~~~~~~~~~ CLK_TCK cc1: all warnings being treated as errors gmake[3]: *** [Makefile:11598: util/libvirt_util_la-vircgroupv1.lo] Error 1 gmake[3]: *** Waiting for unfinished jobs.... rm util/virkeyname-osx.pod util/virkeycode-xtkbd.pod util/virkeycode-atset3.pod util/virkeycode-usb.pod util/virkeycode-qnum.pod util/virkeycode-win32.pod util/virkeyname-linux.pod util/virkeycode-osx.pod util/virkeycode-linux.pod util/virkeycode-atset1.pod util/virkeyname-win32.pod util/virkeycode-atset2.pod gmake[3]: Leaving directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> gmake[2]: *** [Makefile:7013: all] Error 2 gmake[2]: Leaving directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> gmake[1]: *** [Makefile:2111: all-recursive] Error 1 gmake[1]: Leaving directory '<https://ci.centos.org/job/libvirt+mingw32-build/systems=libvirt-fedora-rawh…'> gmake: *** [Makefile:2004: all] Error 2 Build step 'Execute shell' marked build as failure

1 8

[Libvirt-ci] Still Failing: libvirt/libvirt#1725 (master - b7ccd07)
by Travis CI 27 Sep '18

27 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1725 Status: Still Failing Duration: 20 mins and 44 secs Commit: b7ccd07 (master) Author: Jiri Denemark Message: virsh: Require explicit --domain for domxml-to-native The domxml-to-native virsh command accepts either --xml or --domain option followed by a file or domain name respectively. The --domain option is documented as required, which means an argument with no option is treated as --xml. Commit v4.3.0-127-gd86531daf2 broke this by making --domain optional and thus an argument with no option was treated as --domain. https://bugzilla.redhat.com/show_bug.cgi?id=1633077 Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/e94415d5a578...b7ccd0757de7 View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433606503?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Still Failing: libvirt/libvirt#1724 (master - e94415d)
by Travis CI 26 Sep '18

26 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1724 Status: Still Failing Duration: 19 mins and 50 secs Commit: e94415d (master) Author: Ján Tomko Message: tests: libxl: skip tests with nested_hvm Commit 95d19cd unconditionally adjusted the tests to account for the conditional move of the nested_hvm setting location. Run the affected tests only for the new setup (witnessed by LIBXL_HAVE_BUILDINFO_NESTED_HVM). Signed-off-by: Ján Tomko <jtomko(a)redhat.com> Reviewed-by: Erik Skultety <eskultet(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/79e0e62e78ea...e94415d5a578 View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433546256?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Still Failing: libvirt/libvirt#1723 (master - 79e0e62)
by Travis CI 26 Sep '18

26 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1723 Status: Still Failing Duration: 18 mins and 50 secs Commit: 79e0e62 (master) Author: Lin Ma Message: qemu: Remove network type limitation for qemuARPGetInterfaces Let's ignore the checking of interface type when we call the function qemuARPGetInterfaces to get IP from host's arp table. Signed-off-by: Lin Ma <lma(a)suse.com> Reviewed-by: Chen Hanxiao <chenhanxiao(a)gmail.com> View the changeset: https://github.com/libvirt/libvirt/compare/cd33eaa2513a...79e0e62e78ea View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433493747?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Failed: libvirt/libvirt#1722 (v4.8.0-rc1 - cd33eaa)
by Travis CI 26 Sep '18

26 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1722 Status: Failed Duration: 18 mins and 17 secs Commit: cd33eaa (v4.8.0-rc1) Author: Michal Privoznik Message: security: Don't try to lock NULL paths It may happen that in the list of paths/disk sources to relabel there is a disk source. If that is the case, the path is NULL. In that case, we shouldn't try to lock the path. It's likely a network disk anyway and therefore there is nothing to lock. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> Reviewed-by: Erik Skultety <eskultet(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/v4.8.0-rc1 View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433485562?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Still Failing: libvirt/libvirt#1721 (master - cd33eaa)
by Travis CI 26 Sep '18

26 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1721 Status: Still Failing Duration: 17 mins and 49 secs Commit: cd33eaa (master) Author: Michal Privoznik Message: security: Don't try to lock NULL paths It may happen that in the list of paths/disk sources to relabel there is a disk source. If that is the case, the path is NULL. In that case, we shouldn't try to lock the path. It's likely a network disk anyway and therefore there is nothing to lock. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> Reviewed-by: Erik Skultety <eskultet(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/9580c091635d...cd33eaa2513a View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433414202?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Still Failing: libvirt/libvirt#1720 (master - 9580c09)
by Travis CI 26 Sep '18

26 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1720 Status: Still Failing Duration: 18 mins and 31 secs Commit: 9580c09 (master) Author: Marc Hartmayer Message: virdbus: Use the mnemonic macros for dbus_bool_t values Use the mnemonic macros of libdbus for 1 (TRUE) and 0 (FALSE). Signed-off-by: Marc Hartmayer <mhartmay(a)linux.ibm.com> Reviewed-by: Bjoern Walk <bwalk(a)linux.ibm.com> Reviewed-by: Boris Fiuczynski <fiuczy(a)linux.ibm.com> Reviewed-by: Stefan Zimmermann <stzi(a)linux.ibm.com> Reviewed-by: John Ferlan <jferlan(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/e6d77a75c4bf...9580c091635d View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433213453?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Still Failing: libvirt/libvirt#1719 (master - e6d77a7)
by Travis CI 26 Sep '18

26 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1719 Status: Still Failing Duration: 18 mins and 31 secs Commit: e6d77a7 (master) Author: Jiri Denemark Message: qemu: Avoid duplicate resume events and state changes The only place where VIR_DOMAIN_EVENT_RESUMED should be generated is the RESUME event handler to make sure we don't generate duplicate events or state changes. In the worse case the duplicity can revert or cover changes done by other event handlers. For example, after QEMU sent RESUME, BLOCK_IO_ERROR, and STOP events we could happily mark the domain as running and report VIR_DOMAIN_EVENT_RESUMED to registered clients. https://bugzilla.redhat.com/show_bug.cgi?id=1612943 Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com> Reviewed-by: John Ferlan <jferlan(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/7882c6eca53f...e6d77a75c4bf View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433098800?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

[Libvirt-ci] Still Failing: libvirt/libvirt#1718 (master - 7882c6e)
by Travis CI 26 Sep '18

26 Sep '18

Build Update for libvirt/libvirt ------------------------------------- Build: #1718 Status: Still Failing Duration: 19 mins and 18 secs Commit: 7882c6e (master) Author: Mark Asselstine Message: lxc_monitor: Avoid AB / BA lock race A deadlock situation can occur when autostarting a LXC domain 'guest' due to two threads attempting to take opposing locks while holding opposing locks (AB BA problem). Thread A takes and holds the 'vm' lock while attempting to take the 'client' lock, meanwhile, thread B takes and holds the 'client' lock while attempting to take the 'vm' lock. The potential for this can be seen as follows: Thread A: virLXCProcessAutostartDomain (takes vm lock) --> virLXCProcessStart --> virLXCProcessConnectMonitor --> virLXCMonitorNew --> virNetClientSetCloseCallback (wants client lock) Thread B: virNetClientIncomingEvent (takes client lock) --> virNetClientIOHandleInput --> virNetClientCallDispatch --> virNetClientCallDispatchMessage --> virNetClientProgramDispatch --> virLXCMonitorHandleEventInit --> virLXCProcessMonitorInitNotify (wants vm lock) Since these threads are scheduled independently and are preemptible it is possible for the deadlock scenario to occur where each thread locks their first lock but both will fail to get their second lock and just spin forever. You get something like: virLXCProcessAutostartDomain (takes vm lock) --> virLXCProcessStart --> virLXCProcessConnectMonitor --> virLXCMonitorNew <...> virNetClientIncomingEvent (takes client lock) --> virNetClientIOHandleInput --> virNetClientCallDispatch --> virNetClientCallDispatchMessage --> virNetClientProgramDispatch --> virLXCMonitorHandleEventInit --> virLXCProcessMonitorInitNotify (wants vm lock but spins) <...> --> virNetClientSetCloseCallback (wants client lock but spins) Neither thread ever gets the lock it needs to be able to continue while holding the lock that the other thread needs. The actual window for preemption which can cause this deadlock is rather small, between the calls to virNetClientProgramNew() and execution of virNetClientSetCloseCallback(), both in virLXCMonitorNew(). But it can be seen in real world use that this small window is enough. By moving the call to virNetClientSetCloseCallback() ahead of virNetClientProgramNew() we can close any possible chance of the deadlock taking place. There should be no other implications to the move since the close callback (in the unlikely event was called) will spin on the vm lock. The remaining work that takes place between the old call location of virNetClientSetCloseCallback() and the new location is unaffected by the move. Signed-off-by: Mark Asselstine <mark.asselstine(a)windriver.com> Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> View the changeset: https://github.com/libvirt/libvirt/compare/65ba48d26745...7882c6eca53f View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/433006111?utm_medium=notificat… -- You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0