[OSSTEST] stop allowing libvirt failures (Was: Re: [xen-unstable-smoke test] 78436: tolerable FAIL - PUSHED)

Commit Message

Ian Campbell Jan. 18, 2016, 4:06 p.m. UTC

On Mon, 2016-01-18 at 15:44 +0000, osstest service owner wrote:
> flight 78436 xen-unstable-smoke real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/78436/
> 
> Failures :-/ but no regressions.
> 
> Regressions which are regarded as allowable (not blocking):
>  test-amd64-amd64-libvirt      6 xen-boot                  fail REGR. vs. 78183

Well that ain't right...

8<---------

From 13e31f297f9f5e95d510fe7d88dc1c7088fe8c63 Mon Sep 17 00:00:00 2001
From: Ian Campbell <[e-mail]>
Date: Mon, 18 Jan 2016 15:54:15 +0000
Subject: [PATCH] stop allowing libvirt failures

In Feb/Mar 2015 (not long after adding the libvirt tests) we appear to
have added test-@@-libvirt@@ to the set of allowed failures in
response to some issues with libvirtd crashing.

However looking at the history of test-@@-libvirt@@ on all branches
both in the COLO and in Cambridge (which was the production instance
back then) I don't see any evidence that this issue is still ongoing
(which matches my recollection of it having been fixed).

Therefore remove the entries allowing libvirt failures.

This effectively reverts:

00023a5af6ff allow files: Allow all libvirt test failures on other branches
83b8c8eafb18 allow.all: Do not regard libvirt guest start failures as regressions

Signed-off-by: Ian Campbell <[e-mail]>
---
 allow.all     | 1 -
 allow.libvirt | 1 -
 2 files changed, 2 deletions(-)
 delete mode 100644 allow.libvirt

Comments

Ian Campbell Jan. 18, 2016, 4:13 p.m. UTC | #1

On Mon, 2016-01-18 at 16:06 +0000, Ian Campbell wrote:
> On Mon, 2016-01-18 at 15:44 +0000, osstest service owner wrote:
> > flight 78436 xen-unstable-smoke real [real]
> > http://logs.test-lab.xenproject.org/osstest/logs/78436/
> > 
> > Failures :-/ but no regressions.
> > 
> > Regressions which are regarded as allowable (not blocking):
> >  test-amd64-amd64-libvirt      6 xen-boot                  fail REGR.
> > vs. 78183
> 
> Well that ain't right...

WRT the failure of godello1 here http://logs.test-lab.xenproject.org/osstes
t/results/host/godello1.html looks fine, just that one boot failure which
from the serial looks like a stall during boot:

Jan 18 13:49:54.637028 (XEN) TSC deadline timer enabled
Jan 18 13:49:54.853035 (XEN) Platform timer is 14.318MHz HPET
Jan 18 13:49:55.013014 Jan 18 13:55:41.658594 <client 0x1c1b400 connected - now 1 clients>
(XEN) Allocated console ring of 32
Jan 18 13:55:41.673404 <client 0x1c1b400 disconnected - now 0 clients>
 KiB.
Jan 18 13:55:41.681035 (XEN) *** Serial input -> DOM0 (type 'CTRL-x' three times to switch input to Xen)
Jan 18 13:55:41.689042 (XEN) microcode: CPU0 updated from revision 0x17 to 0x1c, date = 2014-07-03 
Jan 18 13:55:41.697012 (XEN) mwait-idle: MWAIT substates: 0x42120
Jan 18 13:55:41.705006 (XEN) mwait-idle: v0.4 model 0x3c

Ian.

Ian Jackson Jan. 19, 2016, 1:44 p.m. UTC | #2

Ian Campbell writes ("[PATCH OSSTEST] stop allowing libvirt failures (Was: Re: [xen-unstable-smoke test] 78436: tolerable FAIL - PUSHED)"):
> On Mon, 2016-01-18 at 15:44 +0000, osstest service owner wrote:
> > flight 78436 xen-unstable-smoke real [real]
> > http://logs.test-lab.xenproject.org/osstest/logs/78436/
> > 
> > Failures :-/ but no regressions.
> > 
> > Regressions which are regarded as allowable (not blocking):
> >  test-amd64-amd64-libvirt      6 xen-boot                  fail REGR. vs. 78183
> 
> Well that ain't right...

Acked-by: Ian Jackson <[e-mail]>

Patch

diff --git a/allow.all b/allow.all
index 8060595..785d0b2 100644
--- a/allow.all
+++ b/allow.all
@@ -2,7 +2,6 @@  test-@@-rtds@@
 build-@@                        logs-capture@@
 test-@@-pcipt@@
 test-@@-qemuu-@@		guest-localmigrate
-test-@@-libvirt@@
 test-@@-rumpuserxen@@		rumpuserxen-demo-xenstorels/xenstorels
 test-@@-win7-@@			guest-stop
 test-@@-rumpuserxen-@@		rumpuserxen-demo-xenstorels/xenstorels.repeat
diff --git a/allow.libvirt b/allow.libvirt
deleted file mode 100644
index abc7669..0000000
--- a/allow.libvirt
+++ /dev/null
@@ -1 +0,0 @@ 
-!test-@@-libvirt@@