diff mbox

[libvirt,test] 107696: regressions - FAIL

Message ID f4a34799-5ded-4027-182a-7b87d06af035@suse.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jim Fehlig May 1, 2017, 10:58 p.m. UTC 
On 04/26/2017 02:34 PM, osstest service owner wrote:
> flight 107696 libvirt real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/107696/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be run:
>  build-arm64-libvirt           5 libvirt-build            fail REGR. vs. 107640
>  build-i386-libvirt            5 libvirt-build            fail REGR. vs. 107640
>  build-amd64-libvirt           5 libvirt-build            fail REGR. vs. 107640
>  build-armhf-libvirt           5 libvirt-build            fail REGR. vs. 107640

I see the bisector has already fingered libvirt commit 02fb15fb, which added a 
new submodule to libvirt.git. I'm an osstest noob, but took a stab at fixing 
this with the attached patch.

Regards,
Jim

Comments

Wei Liu May 2, 2017, 9:19 a.m. UTC | #1 
On Mon, May 01, 2017 at 04:58:27PM -0600, Jim Fehlig wrote:
> On 04/26/2017 02:34 PM, osstest service owner wrote:
> > flight 107696 libvirt real [real]
> > http://logs.test-lab.xenproject.org/osstest/logs/107696/
> > 
> > Regressions :-(
> > 
> > Tests which did not succeed and are blocking,
> > including tests which could not be run:
> >  build-arm64-libvirt           5 libvirt-build            fail REGR. vs. 107640
> >  build-i386-libvirt            5 libvirt-build            fail REGR. vs. 107640
> >  build-amd64-libvirt           5 libvirt-build            fail REGR. vs. 107640
> >  build-armhf-libvirt           5 libvirt-build            fail REGR. vs. 107640
> 
> I see the bisector has already fingered libvirt commit 02fb15fb, which added
> a new submodule to libvirt.git. I'm an osstest noob, but took a stab at
> fixing this with the attached patch.
> 

Thanks. I think Ian already pushed a fix.

Wei.
Ian Jackson May 2, 2017, 10:28 a.m. UTC | #2 
Jim Fehlig writes ("Re: [Xen-devel] [libvirt test] 107696: regressions - FAIL"):
> On 04/26/2017 02:34 PM, osstest service owner wrote:
> > flight 107696 libvirt real [real]
> > http://logs.test-lab.xenproject.org/osstest/logs/107696/
> >
> > Regressions :-(
> >
> > Tests which did not succeed and are blocking,
> > including tests which could not be run:
> >  build-arm64-libvirt           5 libvirt-build        fail REGR. vs. 107640
> >  build-i386-libvirt            5 libvirt-build        fail REGR. vs. 107640
> >  build-amd64-libvirt           5 libvirt-build        fail REGR. vs. 107640
> >  build-armhf-libvirt           5 libvirt-build        fail REGR. vs. 107640
> 
> I see the bisector has already fingered libvirt commit 02fb15fb,
> which added a new submodule to libvirt.git. I'm an osstest noob, but
> took a stab at fixing this with the attached patch.

Thanks.  Your patch is correct.  In fact, it is identical apart from
formatting to the patch I pushed to osstest pretest last week.

Unfortunately the persistent Windows heisenbugs mean that that osstest
change didn't get a push.  I will force push it.

Thanks,
Ian.
Jim Fehlig May 2, 2017, 2:33 p.m. UTC | #3 
On 05/02/2017 04:28 AM, Ian Jackson wrote:
> Jim Fehlig writes ("Re: [Xen-devel] [libvirt test] 107696: regressions - FAIL"):
>> On 04/26/2017 02:34 PM, osstest service owner wrote:
>>> flight 107696 libvirt real [real]
>>> http://logs.test-lab.xenproject.org/osstest/logs/107696/
>>>
>>> Regressions :-(
>>>
>>> Tests which did not succeed and are blocking,
>>> including tests which could not be run:
>>>  build-arm64-libvirt           5 libvirt-build        fail REGR. vs. 107640
>>>  build-i386-libvirt            5 libvirt-build        fail REGR. vs. 107640
>>>  build-amd64-libvirt           5 libvirt-build        fail REGR. vs. 107640
>>>  build-armhf-libvirt           5 libvirt-build        fail REGR. vs. 107640
>>
>> I see the bisector has already fingered libvirt commit 02fb15fb,
>> which added a new submodule to libvirt.git. I'm an osstest noob, but
>> took a stab at fixing this with the attached patch.
>
> Thanks.  Your patch is correct.  In fact, it is identical apart from
> formatting to the patch I pushed to osstest pretest last week.

Ah, I didn't see your change. Sorry for the noise.

> Unfortunately the persistent Windows heisenbugs mean that that osstest
> change didn't get a push.  I will force push it.

Thanks!

Regards,
Jim
diff mbox

Patch

From e1c9d55c046b6b826af95d3bb6cfaf63715f2923 Mon Sep 17 00:00:00 2001
From: Jim Fehlig <jfehlig@suse.com>
Date: Mon, 1 May 2017 16:45:10 -0600
Subject: [PATCH] osstest: add keycodemapdb to list of libvirt git submodules

libvirt commit 02fb15fb added a git submodule for keycode map,
doing away with the old, stale keymaps.csv copied from GTK-VNC.
This submodule needs to be handled by osstest when checking out
libvirt.

Signed-off-by: Jim Fehlig <jfehlig@suse.com>
---
 ts-libvirt-build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ts-libvirt-build b/ts-libvirt-build
index 714402b..7bd5f80 100755
--- a/ts-libvirt-build
+++ b/ts-libvirt-build
@@ -26,7 +26,7 @@  tsreadconfig();
 selectbuildhost(\@ARGV);
 builddirsprops();
 
-our %submodmap = qw(gnulib gnulib);
+our %submodmap = qw(gnulib gnulib keycodemapdb keycodemapdb);
 our $submodules;
 
 sub libvirtd_init ();
-- 
2.11.0