|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [libvirt test] 86625: regressions - FAIL
flight 86625 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/86625/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-libvirt 5 libvirt-build fail REGR. vs. 86536 Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-xsm 1 build-check(1) blocked n/a test-armhf-armhf-libvirt 1 build-check(1) blocked n/a test-armhf-armhf-libvirt-qcow2 1 build-check(1) blocked n/a test-armhf-armhf-libvirt-raw 1 build-check(1) blocked n/a test-amd64-i386-libvirt 12 migrate-support-check fail never pass test-amd64-i386-libvirt-xsm 12 migrate-support-check fail never pass test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 10 migrate-support-check fail never pass test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 10 migrate-support-check fail never pass test-amd64-amd64-libvirt 12 migrate-support-check fail never pass test-amd64-amd64-libvirt-vhd 11 migrate-support-check fail never pass test-amd64-amd64-libvirt-xsm 12 migrate-support-check fail never pass version targeted for testing: libvirt 2dabe2e03e2ebd7ef28f2c61a019330a172b43a7 baseline version: libvirt 9a0c7f5f834185db9017c34aabc03ad99cf37bed Last test of basis 86536 2016-03-18 04:25:19 Z 1 days Testing same since 86625 2016-03-19 04:23:22 Z 0 days 1 attempts ------------------------------------------------------------ People who touched revisions under test: Cole Robinson <crobinso@xxxxxxxxxx> Jim Fehlig <jfehlig@xxxxxxxx> John Ferlan <jferlan@xxxxxxxxxx> Martin Kletzander <mkletzan@xxxxxxxxxx> Michal Privoznik <mprivozn@xxxxxxxxxx> jobs: build-amd64-xsm pass build-armhf-xsm pass build-i386-xsm pass build-amd64 pass build-armhf pass build-i386 pass build-amd64-libvirt pass build-armhf-libvirt fail build-i386-libvirt pass build-amd64-pvops pass build-armhf-pvops pass build-i386-pvops pass test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm pass test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm pass test-amd64-amd64-libvirt-xsm pass test-armhf-armhf-libvirt-xsm blocked test-amd64-i386-libvirt-xsm pass test-amd64-amd64-libvirt pass test-armhf-armhf-libvirt blocked test-amd64-i386-libvirt pass test-amd64-amd64-libvirt-pair pass test-amd64-i386-libvirt-pair pass test-armhf-armhf-libvirt-qcow2 blocked test-armhf-armhf-libvirt-raw blocked test-amd64-amd64-libvirt-vhd pass ------------------------------------------------------------ sg-report-flight on osstest.test-lab.xenproject.org logs: /home/logs/logs images: /home/logs/images Logs, config files, etc. are available at http://logs.test-lab.xenproject.org/osstest/logs Explanation of these reports, and of osstest in general, is at http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README.email;hb=master http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README;hb=master Test harness code can be found at http://xenbits.xen.org/gitweb?p=osstest.git;a=summary Not pushing. ------------------------------------------------------------ commit 2dabe2e03e2ebd7ef28f2c61a019330a172b43a7 Author: Cole Robinson <crobinso@xxxxxxxxxx> Date: Wed Jan 6 15:44:30 2016 -0500 domain: Remove controller/net address whitelists Judging by how the whitelist has skewed quite far from the original error message, I think it's better to just drop these. If someone wants to revive this check I suggest implementing it on a per-HV driver basis with PostParse callbacks. commit d77ffb6876e87a5c6f4c74c49cf0d89ade4f8326 Author: Martin Kletzander <mkletzan@xxxxxxxxxx> Date: Tue Mar 15 12:22:03 2016 +0100 nodedev: Expose PCI header type If we expose this information, which is one byte in every PCI config file, we let all mgmt apps know whether the device itself is an endpoint or not so it's easier for them to decide whether such device can be passed through into a VM (endpoint) or not (*-bridge). Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1317531 Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx> commit 0d8f45246a74b627e62eccef2bb19068690312e2 Author: Martin Kletzander <mkletzan@xxxxxxxxxx> Date: Tue Mar 15 13:13:48 2016 +0100 nodedev: Indent PCI express for future fix Best viewed with '-w' as this is just an adjustment for future patch to be readable without that. Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx> commit 5bcd96f5317bfdb66fc66ecf1103cd40b5666a2b Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Thu Mar 3 15:29:51 2016 +0100 tests: Introduce nsslinktest The only purpose of this test is to catch possible linking problems with libnss_libvirt.so.2. One of the problems I faced was that the NSS plugin was unloaded immediately after it got loaded and the name resolution process continued with next configured option. Without any error. It was very hard to debug why until I created this simple test and found out immediately that there were some symbols missing. The reason why problem was not caught in nsstest is that in the test we want to use all the fancy stuff and therefore link it with libvirt.la. So even if there's a symbol missing in the NSS plugin it will be found in the libvirt.la. But even after I resolved the issue we still need this test because files the NSS plugin is built from are still live (mostly those under utils/ dir). So as they change new symbol might be required which would render the NSS plugin unusable. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 38e32d4ac134aa6da411b0093e788bce35775d09 Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Mon Feb 15 14:02:05 2016 +0100 nss: Introduce a test A small test to see how is the nss module working. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 917038c110b67648c112f588d2a130ba33ed0bf9 Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Sun Feb 14 08:13:54 2016 +0100 Implement _nss_libvirt_gethostbyname4_r This function is a different beast compared to previous ones. But yet again, nothing surprising is happening here. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 7dbcb26f7f67b9ff2bee47e6144763a3d729717e Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Sun Feb 14 11:38:37 2016 +0100 nss: Implement _nss_libvirt_gethostbyname3_r The implementation is pretty straightforward. Moreover, because of the nature of things, gethostbyname_r and gethostbyname2_r can be implemented at the same time too. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 8054706c3fad02e044297bc791cef1f92aee7e74 Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Tue Feb 16 09:41:30 2016 +0100 libvirt.spec.in: Introduce libvirt-nss package Lets put the NSS module into its own package. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 859cb18d5401ceaf678dc4b283b24b9af7f8f827 Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Sat Feb 13 09:27:50 2016 +0100 Initial support for NSS plugin skeleton Name Service Switch is a glibc feature responsible for many things. Translating domain names into IP addresses and vice versa is just one of them. However, currently it's the only functionality that this commit is tickling. Well, in this commit the plugin skeleton is introduced. Implementation to come in next patches. Because of the future testing, where the implementation is to be linked with a test, this needs to go into static library. Linking a program with an .so statically is not portable. Therefore a dummy libnss_libvirt_impl library is being introduced too. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 1ca59d89c6b7503fe065fb75d717e1487aea5b84 Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Tue Mar 15 16:35:23 2016 +0100 virsocketaddr: Introduce virSocketAddrSetIPv6Addr This is a missing counterpart for virSocketAddrSetIPv4Addr() and is going to be needed later in the tests. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 4122137871cbf604451d53b0063192f8ee2592e0 Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Tue Mar 15 16:49:37 2016 +0100 virLeaseReadCustomLeaseFile: Allow server_duid to be NULL This function is going to be used later in such context where the argument makes no sense. Teach this function to cope with that instead of the caller having to deal with passing some dummy argument. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit fd9514f8d27880014e08f7984dfa31df6a6adb0f Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Sun Feb 14 07:50:12 2016 +0100 Export virLease* functions for leases file handling These functions are going to be reused very shortly. So instead of duplicating the code, lets move them into utils module. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit adddaff1836f928b14865771294bab0b1eedd01c Author: Jim Fehlig <jfehlig@xxxxxxxx> Date: Mon Mar 14 16:35:02 2016 -0600 tests: add schema test for default cache mode None of the existing domXML configs under tests/* specify a default cache mode since default generally means "use the hypervisor default" and is left unset by the various hypervisor drivers. Add a config to tests/domainschemadata that specifies cache='default'. commit 1a01e8792c70fde251fd422b46e1062cdba91794 Author: Jim Fehlig <jfehlig@xxxxxxxx> Date: Mon Mar 14 16:27:41 2016 -0600 tests: add xlconfigdata to domainschematests Include the XML files under tests/xlconfigdata in the domain schema tests commit efb5e46b6c2ee157fee90bfb65ca60dce13056a2 Author: John Ferlan <jferlan@xxxxxxxxxx> Date: Thu Mar 17 07:29:23 2016 -0400 conf: Format disk pool part_separator attribute for running pool Commit id '4f846170' added printing of a new field 'part_separator'; however, neglected to do so when there was an "freeExtent" defined for the device (as there would be when the disk pool was started). This patch adjusts the logic to appropriately format the device path and if there the part_separator attribute. commit 1019bf1445318ab94481b4fabe736be9e77946f2 Author: John Ferlan <jferlan@xxxxxxxxxx> Date: Tue Mar 15 07:15:30 2016 -0400 qemu: Move last error save/restore to qemuBuildNetCommandLine Commit 'ef2ab8fd' moved just the virDomainConfNWFilterTeardown and left the logic to save/restore the current error essentially doing nothing in the error path for qemuBuildCommandLine. So move it to where it was meant to be. Although the original code would reset the filter on command creation errors after building the network command portion and commit 'ef2ab8fd' altered that logic, the teardown is called during qemuProcessStop from virDomainConfVMNWFilterTeardown and that code has the save/restore last error logic, so just allow that code to handle the teardown rather than running it twice. The qemuProcessStop would be called in the failure path of qemuBuildCommandLine. commit 363b401f9473dee21c152866fc158d231ddcb6dd Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Thu Mar 17 13:30:42 2016 +0100 tests: Set PATH in each test Currently we spawn couple of binaries in our test suite. Moreover, we provide some spoofed versions of system binaries hoping that those will be executed instead of the system ones. For instance, for testing SSH socket we have written our own ssh binary for producing predictable results. We certainly don't want to execute the system ssh binary. However, in order to prefer our binaries over system ones, we need to set PATH environment variable. But this is done only at the Makefile level. So if anybody runs a test by hand that expects our spoofed binary, the test ends up executing real system binaries. This is not good. In fact, it's terribly wrong. The fix lies in a small trick - putting our build directory at the beginning of the PATH environment variable in each test. Hopefully, since every test has this VIRT_TEST_MAIN* wrapper, we can fix this at a single place. Moreover, while this removes setting PATH for our tests written in bash, it's safe as we are not calling anything ours that would require PATH change there. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> commit 865764de065585978d70624cfe4a4f960180879d Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Thu Mar 17 11:44:01 2016 +0100 Drop paths.h include We include the file in plenty of places. This is mostly due to historical reasons. The only place that needs something from the header file is storage_backend_fs which opens _PATH_MOUNTED. But it gets the file included indirectly via mntent.h. At no other place in our code we need _PATH_.*. Drop the include and configure check then. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |