|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [libvirt test] 132490: regressions - FAIL
flight 132490 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/132490/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-raw 15 guest-start/debian.repeat fail REGR. vs. 132469 Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 14 saverestore-support-check fail like 132469 test-armhf-armhf-libvirt-raw 13 saverestore-support-check fail like 132469 test-amd64-i386-libvirt-xsm 13 migrate-support-check fail never pass test-amd64-i386-libvirt 13 migrate-support-check fail never pass test-amd64-amd64-libvirt-xsm 13 migrate-support-check fail never pass test-amd64-amd64-libvirt 13 migrate-support-check fail never pass test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 11 migrate-support-check fail never pass test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 11 migrate-support-check fail never pass test-amd64-amd64-libvirt-vhd 12 migrate-support-check fail never pass test-armhf-armhf-libvirt 13 migrate-support-check fail never pass test-armhf-armhf-libvirt-raw 12 migrate-support-check fail never pass version targeted for testing: libvirt 001495909b0c11e21723671f541cae66912a40f4 baseline version: libvirt 429f5454d570b5fca4b40338502bea724e31db99 Last test of basis 132469 2019-01-25 05:11:45 Z 2 days Testing same since 132490 2019-01-26 06:39:12 Z 1 days 1 attempts ------------------------------------------------------------ People who touched revisions under test: Eric Blake <eblake@xxxxxxxxxx> Ján Tomko <jtomko@xxxxxxxxxx> Laine Stump <laine@xxxxxxxxx> Michal Privoznik <mprivozn@xxxxxxxxxx> Thomas Huth <thuth@xxxxxxxxxx> jobs: build-amd64-xsm pass build-i386-xsm pass build-amd64 pass build-armhf pass build-i386 pass build-amd64-libvirt pass build-armhf-libvirt pass 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-amd64-i386-libvirt-xsm pass test-amd64-amd64-libvirt pass test-armhf-armhf-libvirt pass test-amd64-i386-libvirt pass test-amd64-amd64-libvirt-pair pass test-amd64-i386-libvirt-pair pass test-armhf-armhf-libvirt-raw fail 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 001495909b0c11e21723671f541cae66912a40f4 Author: Laine Stump <laine@xxxxxxxxx> Date: Wed Jan 9 18:45:49 2019 -0500 docs: add forgotten mentions of forward mode "open" A couple places in the docs didn't get updated when the forward mode "open" was added. Signed-off-by: Laine Stump <laine@xxxxxxxxx> Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx> commit 43be65a4817fcd1583776368999c034540dfaa57 Author: Laine Stump <laine@xxxxxxxxx> Date: Wed Jan 9 19:42:41 2019 -0500 network: remove stale function networkMigrateStateFiles was added nearly 5 years ago when the network state directory was moved from /var/lib/libvirt to /var/run/libvirt just prior to libvirt-1.2.4). It was only required to maintain proper state information for networks that were active during an upgrade that didn't involve rebooting the host. At this point the likelyhood of anyone upgrading their libvirt from pre-1.2.4 directly to 5.0.0 or later *without rebooting the host* is probably so close to 0 that no properly informed bookie would take *any* odds on it happening, so it seems appropriate to remove this pointless code. Signed-off-by: Laine Stump <laine@xxxxxxxxx> Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx> commit 484370dcf1c927ee73b0f278d0ff6065747a9a42 Author: Eric Blake <eblake@xxxxxxxxxx> Date: Thu Jan 24 21:32:55 2019 -0600 virjson: add convenience wrapper for appending string to array Upcoming patches need an array of strings for use in QMP block-dirty-bitmap-merge. A convenience wrapper cuts down on the verbosity of creating the array, similar to the existing virJSONValueObjectAppendString(). Signed-off-by: Eric Blake <eblake@xxxxxxxxxx> Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> commit 4ea5a41e16f76358d2bdce194dce0ba9bc5f0261 Author: Eric Blake <eblake@xxxxxxxxxx> Date: Thu Jan 24 21:28:23 2019 -0600 virjson: always raise vir error on append failures A function that returns -1 for multiple possible failures, but only raises a libvirt error for some of those failures, can be hard to use correctly. Yet both of our JSON object/array appenders fall in that pattern. True, the silent errors represent coding bugs that none of the callers should ever trigger, while the noisy errors represent memory failures that can happen anywhere, so we happened to never end up failing without an error. But it is better to either use the _QUIET memory allocation variants, and make callers decide to report failure; or make all failure paths noisy. This patch takes the latter approach. Signed-off-by: Eric Blake <eblake@xxxxxxxxxx> Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> commit 2707d4f8bc7a65dea13b47422f0fe4c1309f9dc2 Author: Michal Privoznik <mprivozn@xxxxxxxxxx> Date: Tue Jan 22 12:23:07 2019 +0100 tools: Document completer callback Strictly speaking, this should go near vshCompleter typedef declaration. However, I find it more useful near actual completer implementations. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx> commit 49ec5769c9dfdfbaabe939d0851bda95364a05d6 Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Wed Jan 16 08:47:06 2019 +0100 qemuBuildControllersCommandLine: use i instead of j as the counter Now that the nested loop is gone. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit a6d3357c8b9eb3b08f17703d7154cd8c23e60bdc Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Wed Jan 16 08:46:29 2019 +0100 rename qemuBuildControllerDevCommandLine Use qemuBuildControllersCommandLine since it builds the command line for (nearly) all controllers, not just one. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit fc1c16bccd4deb86848a3f37ccb55831ed8ce2f2 Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 17:59:17 2019 +0100 qemuBuildControllersByTypeCommandLine: free devstr in the cleanup section Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit c7e03df8c0f38bb007523c022e628581948e8f01 Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 16:50:22 2019 +0100 qemu: format CCID controllers after USB hubs Since they go on the USB bus, format them after USB hubs. https://bugzilla.redhat.com/show_bug.cgi?id=1375402 Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit 421e0eeec7773e3c4729d7035dc9219fa71b4421 Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 16:44:41 2019 +0100 qemu: Introduce qemuBuildControllersByTypeCommandLine Now that the inner loop does not require any other variables, it can be easily separated. Apart from reducing the indentation level this will allow it to be called from different code paths. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit 75ecfd252136878ec39eed4915691f61d397c28d Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 16:31:54 2019 +0100 qemuBuildControllerDevStr: remove nusbcontroller argument Now that it's no longer needed, remove the argument. This removes the last helper variable in qemuBuildControllerDevCommandLine. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit 8f5e50e003ece6177634736c2914872d91c71e4f Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 16:28:55 2019 +0100 qemu: separate counting of USB controllers qemuBuildLegacyUSBControllerCommandLine is the only place where we need to count the USB controllers. Count them again instead of keeping track in a variable passed to qemuBuildControllerDevStr. This removes the need for another variable in the loop in qemuBuildControllerDevCommandLine. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit dab3d5d350a840621f79e390abe42a1fea13d750 Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 16:15:39 2019 +0100 qemu: separate counting of legacy USB controllers Count them in qemuBuildLegacyUSBControllerCommandLine to remove yet another variable accessed from the loop in qemuBuildControllerDevCommandLine. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit 95049d70540ef2c9cc5bcfd96f33d28a00fd95dd Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 16:08:48 2019 +0100 Add qemuBuildDomainForbidLegacyUSBController Shorten some long conditions. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit 79c7cbaf46682c41cc4b6967868fcc79468f1a35 Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 15:46:39 2019 +0100 qemu: exit early if USB_CONTROLLER_MODEL_NONE is present This removes the need to mark it in the 'usbcontroller' variable. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit a98c0a2fef11fc2206f8ee151d518cfdad535d6a Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Tue Jan 15 15:33:10 2019 +0100 qemu: move out legacy USB controller formatting Move out the code formatting "-usb" on the QEMU command line. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit c2a8256991d772323a3df08c33b012bf12d62645 Author: Ján Tomko <jtomko@xxxxxxxxxx> Date: Mon Jan 21 15:49:23 2019 +0100 virPortAllocatorSetUsed: ignore port 0 Similar to what commit 86dba8f3 did for virPortAllocatorRelease, ignore port 0 in virPortAllocatorSetUsed. For all the reasonable use cases the callers already check that the port is non-zero, however if the port from the XML overflows unsigned short and turns into 0, it can be set as used by virPortAllocatorSetUsed but not released by virPortAllocatorRelease. Also skip port '0' in virPortAllocatorSetUsed to make this behavior symmetric. The serenity was disturbed by commit 5dbda5e9 which started using virPortAllocatorRelease instead of virPortAllocatorSetUsed (false). https://bugzilla.redhat.com/show_bug.cgi?id=1591645 Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> commit 872b15dea6623490c223b0c8081e14f280459a60 Author: Thomas Huth <thuth@xxxxxxxxxx> Date: Fri Jan 25 10:50:28 2019 +0100 docs/governance: Clarify the version number of the LGPL There is no "GNU Lesser General Public License, version 2", only version 2.1 and later. In "version 2", the license was still called "Library" instead of "Lesser". So assume that version 2.1 is meant here. Signed-off-by: Thomas Huth <thuth@xxxxxxxxxx> Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> commit 803a73980a714ef207294e37db4115bba0f0a7d8 Author: Thomas Huth <thuth@xxxxxxxxxx> Date: Fri Jan 25 10:50:27 2019 +0100 tools/virt-xml-validate: Fix GPL information The tools/virt-xml-validate.in file is licensed under the terms of the GPL, but then says "You should have received a copy of the GNU *Lesser* General Public License". Thus scratch the "Lesser" here. Signed-off-by: Thomas Huth <thuth@xxxxxxxxxx> Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> commit 70d9c1a768e56812d2bc3ed6ddb286467242ada3 Author: Thomas Huth <thuth@xxxxxxxxxx> Date: Fri Jan 25 10:50:26 2019 +0100 bootstrap.conf: Fix LGPL information The bootstrap.conf is licensed under the terms of the LGPL, but then suggests to "See the GNU General Public License for more details". That should be the "GNU Lesser General Public License" instead, of course. Signed-off-by: Thomas Huth <thuth@xxxxxxxxxx> Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |