All of lore.kernel.org
 help / color / mirror / Atom feed
* [libvirt test] 107696: regressions - FAIL
@ 2017-04-26 20:34 osstest service owner
  2017-05-01 22:58 ` Jim Fehlig
  0 siblings, 1 reply; 5+ messages in thread
From: osstest service owner @ 2017-04-26 20:34 UTC (permalink / raw)
  To: xen-devel, osstest-admin

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

Tests which did not succeed, but are not blocking:
 test-amd64-amd64-libvirt-vhd  1 build-check(1)               blocked  n/a
 test-arm64-arm64-libvirt-xsm  1 build-check(1)               blocked  n/a
 test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 1 build-check(1) blocked n/a
 test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 1 build-check(1) blocked n/a
 test-armhf-armhf-libvirt      1 build-check(1)               blocked  n/a
 test-arm64-arm64-libvirt-qcow2  1 build-check(1)               blocked  n/a
 test-amd64-i386-libvirt       1 build-check(1)               blocked  n/a
 test-amd64-amd64-libvirt-pair  1 build-check(1)               blocked  n/a
 test-armhf-armhf-libvirt-raw  1 build-check(1)               blocked  n/a
 test-arm64-arm64-libvirt      1 build-check(1)               blocked  n/a
 test-amd64-i386-libvirt-xsm   1 build-check(1)               blocked  n/a
 test-armhf-armhf-libvirt-xsm  1 build-check(1)               blocked  n/a
 test-amd64-amd64-libvirt-xsm  1 build-check(1)               blocked  n/a
 test-amd64-i386-libvirt-pair  1 build-check(1)               blocked  n/a
 test-amd64-amd64-libvirt      1 build-check(1)               blocked  n/a

version targeted for testing:
 libvirt              e6c3b59c193ec1d17d72b65d460a951227c12a72
baseline version:
 libvirt              5efa7f2a4bf2e316ca74b5baad053a18cffd00b9

Last test of basis   107640  2017-04-25 04:21:01 Z    1 days
Testing same since   107696  2017-04-26 04:20:12 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Andrea Bolognani <abologna@redhat.com>
  Daniel P. Berrange <berrange@redhat.com>
  John Ferlan <jferlan@redhat.com>
  Wang King <king.wang@huawei.com>
  ZhiPeng Lu <lu.zhipeng@zte.com.cn>

jobs:
 build-amd64-xsm                                              pass    
 build-arm64-xsm                                              pass    
 build-armhf-xsm                                              pass    
 build-i386-xsm                                               pass    
 build-amd64                                                  pass    
 build-arm64                                                  pass    
 build-armhf                                                  pass    
 build-i386                                                   pass    
 build-amd64-libvirt                                          fail    
 build-arm64-libvirt                                          fail    
 build-armhf-libvirt                                          fail    
 build-i386-libvirt                                           fail    
 build-amd64-pvops                                            pass    
 build-arm64-pvops                                            pass    
 build-armhf-pvops                                            pass    
 build-i386-pvops                                             pass    
 test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm           blocked 
 test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm            blocked 
 test-amd64-amd64-libvirt-xsm                                 blocked 
 test-arm64-arm64-libvirt-xsm                                 blocked 
 test-armhf-armhf-libvirt-xsm                                 blocked 
 test-amd64-i386-libvirt-xsm                                  blocked 
 test-amd64-amd64-libvirt                                     blocked 
 test-arm64-arm64-libvirt                                     blocked 
 test-armhf-armhf-libvirt                                     blocked 
 test-amd64-i386-libvirt                                      blocked 
 test-amd64-amd64-libvirt-pair                                blocked 
 test-amd64-i386-libvirt-pair                                 blocked 
 test-arm64-arm64-libvirt-qcow2                               blocked 
 test-armhf-armhf-libvirt-raw                                 blocked 
 test-amd64-amd64-libvirt-vhd                                 blocked 


------------------------------------------------------------
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 e6c3b59c193ec1d17d72b65d460a951227c12a72
Author: Daniel P. Berrange <berrange@redhat.com>
Date:   Fri Mar 3 12:43:51 2017 +0000

    Add ability to generate man page describing key code names & values
    
    Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

commit 02fb15fb60274a684497293fecba400e37687d95
Author: Daniel P. Berrange <berrange@redhat.com>
Date:   Thu Mar 2 10:46:53 2017 +0000

    util: switch over to use keycodemapdb GIT submodule
    
    A long time ago we imported the keymaps.csv file from GTK-VNC so we
    can do conversions between keycode sets. Meanwhile lots of bug fixes
    have gone into this CSV file and libvirt hasn't kept in sync. The
    keymaps.csv file and associated generator script has been pulled out
    of GTK-VNC into a dedicated GIT repo for use as a submodule. This
    allows GTK-VNC, SPICE-GTK, QEMU and libvirt to share the same master
    database and tools and pushing updates merely requires a submodule
    commit update as with gnulib.
    
    The test suite is updated to cover some extra boundary conditions.
    
    Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

commit ab54d5f15239940d4e3d142c553d7e55ec0ff13b
Author: John Ferlan <jferlan@redhat.com>
Date:   Mon Apr 24 12:50:12 2017 -0400

    secret: Clean up virSecretObjListExport logic
    
    Shorten the time needed to keep the list lock and alter the cleanup
    path to be more of an error path.
    
    Utilize the the virObjectListFree function to handle the calls for
    virObjectUnref on each list element and the VIR_FREE of the list
    instead of open coding it.
    
    Change the name of the virHashForEach callback to match the name
    of the Export function with the Callback added onto it.
    
    Signed-off-by: John Ferlan <jferlan@redhat.com>

commit 891d0a76b5ad34959c585660a62d13d51d1b8a96
Author: John Ferlan <jferlan@redhat.com>
Date:   Thu Apr 20 11:22:21 2017 -0400

    secret: Have virSecretObjNew return locked object
    
    Rather than have caller need to do it, have the object returned locked.
    
    Signed-off-by: John Ferlan <jferlan@redhat.com>

commit c520f3f160efac3725f9030596a591b6aa6635e8
Author: John Ferlan <jferlan@redhat.com>
Date:   Mon Apr 3 07:47:58 2017 -0400

    secret: Convert virsecretobjs.h to use "newer" formatting style
    
    Alter the prototypes to use the newer formatting style
    
    Signed-off-by: John Ferlan <jferlan@redhat.com>

commit 74face42b653f02e7e02866f296cf696731ea7fa
Author: John Ferlan <jferlan@redhat.com>
Date:   Fri Apr 14 12:35:26 2017 -0400

    secret: Need to set data->error on VIR_ALLOC_N failure
    
    Commit id 'bb1fba629' neglected to set when creating the function.
    
    Signed-off-by: John Ferlan <jferlan@redhat.com>

commit 6b5ac3c71d8472f7e4d5c45ada31ae53a157af03
Author: John Ferlan <jferlan@redhat.com>
Date:   Fri Apr 14 09:24:33 2017 -0400

    test: Remove unnecessary unlocks in cleanup paths
    
    Commit id '865f479da' altered the logic to use a common test*ObjFindByName
    helpers which would lock/unlock the test driver; however, a few cleanup paths
    in that cleanup missed removing the Unlock, so remove it now.
    
    Signed-off-by: John Ferlan <jferlan@redhat.com>

commit fe8f1c8b8650c8ab5e7d27338f4538582651bd14
Author: John Ferlan <jferlan@redhat.com>
Date:   Mon Mar 27 12:47:37 2017 -0400

    remote: Fix possible use-after-free when sending event message
    
    Based upon an idea and some research by Wang King <king.wang@huawei.com>
    and xinhua.Cao <caoxinhua@huawei.com>.
    
    Since we're assigning the 'client' to our callback event lookaside list,
    it's imperative that we grab a reference to the object; otherwise, when
    the object is unref'd during virNetServerProcessClients when it's determined
    that the virNetServerClientIsClosed and the memory is free'd before perhaps
    the object event state callbacks are run.  When a virObjectLock() is run,
    before sending the message the following trace occurs;
    
        #0  0x00007fda223d66d8 in virClassIsDerivedFrom
            (klass=0xdeadbeef, parent=0x7fda24c81b40)
             at util/virobject.c:169
        #1  0x00007fda223d6a1e in virObjectIsClass
            (anyobj=anyobj@entry=0x7fd9e575b400, klass=<optimized out>)
             at util/virobject.c:365
        #2  0x00007fda223d6a44 in virObjectLock
            (anyobj=0x7fd9e575b400)
            at util/virobject.c:317
        #3  0x00007fda22507f71 in virNetServerClientSendMessage
            (client=client@entry=0x7fd9e575b400, msg=msg@entry=0x7fd9ec30de90)
            at rpc/virnetserverclient.c:1422
        #4  0x00007fda230d714d in remoteDispatchObjectEventSend
            (client=0x7fd9e575b400, program=0x7fda24c844e0, procnr=348,
             proc=0x7fda2310e5e0 <xdr_remote_domain_event_callback_tunable_msg>,
             data=0x7ffc3857fdb0)
            at remote.c:3803
        #5  0x00007fda230dd71b in remoteRelayDomainEventTunable
            (conn=<optimized out>, dom=0x7fda27cd7660, params=0x7fda27f3aae0,
             nparams=1,opaque=0x7fd9e6c99e00)
            at remote.c:1033
        #6  0x00007fda224484cb in virDomainEventDispatchDefaultFunc
            (conn=0x7fda27cd0120, event=0x7fda2736ea00, cb=0x7fda230dd610
             <remoteRelayDomainEventTunable>, cbopaque=0x7fd9e6c99e00)
            at conf/domain_event.c:1910
        #7  0x00007fda22446871 in virObjectEventStateDispatchCallbacks
            (callbacks=<optimized out>, callbacks=<optimized out>,
             event=0x7fda2736ea00,state=0x7fda24ca3960)
            at conf/object_event.c:722
        #8  virObjectEventStateQueueDispatch
            (callbacks=0x7fda24c65800, queue=0x7ffc3857fe90, state=0x7fda24ca3960)
            at conf/object_event.c:736
        #9  virObjectEventStateFlush (state=0x7fda24ca3960)
            at conf/object_event.c:814
        #10 virObjectEventTimer (timer=<optimized out>, opaque=0x7fda24ca3960)
            at conf/object_event.c:560
        #11 0x00007fda223ae8b9 in virEventPollDispatchTimeouts ()
            at util/vireventpoll.c:458
        #12 virEventPollRunOnce ()
            at util/vireventpoll.c:654
        #13 0x00007fda223ad1d2 in virEventRunDefaultImpl ()
            at util/virevent.c:314
        #14 0x00007fda225046cd in virNetDaemonRun (dmn=0x7fda24c775c0)
            at rpc/virnetdaemon.c:818
        #15 0x00007fda230d6351 in main (argc=<optimized out>, argv=<optimized out>)
            at libvirtd.c:1623
    
    Signed-off-by: John Ferlan <jferlan@redhat.com>

commit 2033e8cc119454bc4273e8a41e66c899c60ba58b
Author: John Ferlan <jferlan@redhat.com>
Date:   Mon Mar 27 12:47:36 2017 -0400

    daemon: Rework remoteClientFreeFunc cleanup loops into C macro
    
    Rather than 'n' repetitive code segments, let's create a single macro
    which will make the code easier to read.
    
    Signed-off-by: John Ferlan <jferlan@redhat.com>

commit 0023cc039dc1730cb5945fa21df6b067cb6495c6
Author: Wang King <king.wang@huawei.com>
Date:   Mon Apr 24 12:05:25 2017 +0800

    daemon: Remove unnecessary goto error
    
    Freeing the dst is unnecessary if the VIR_STRDUP fails, and therefore
    we need to remove the error label as well.

commit f69340472cf71c11313ad0018c918d702b08d155
Author: Wang King <king.wang@huawei.com>
Date:   Mon Apr 24 12:05:24 2017 +0800

    daemon: Fix domain name leak in error path
    
    Domain name is duplicated in make_nonnull_domain, but not freed when
    virTypedParamsSerialize returns error.

commit c77bc47f43dff2ceb2b95e1aee6fbe1ec9e32fa3
Author: ZhiPeng Lu <lu.zhipeng@zte.com.cn>
Date:   Tue Apr 25 14:16:20 2017 +0800

    qemu: fix argument of virDomainNetGetActualDirectMode
    
    it should be a comparison of modes between new and old devices. So
    the argument of the second virDomainNetGetActualDirectMode should be
    newdev.
    
    Signed-off-by: ZhiPeng Lu <lu.zhipeng@zte.com.cn>

commit 6d53255377b50b7e412350d0795818854738fe2b
Author: Andrea Bolognani <abologna@redhat.com>
Date:   Wed Apr 12 13:09:08 2017 +0200

    autogen.sh: Improve and generalize
    
    The goal is twofold: firstly, we want to extend the script so
    that it can deal with more than a single git submodule, and
    secondly we'd like to reduce the amount of duplicated code.
    Moreover, since we're making heavy changes to the code anyway,
    we might as well make sure it follows a somewhat consistent
    coding style too.
    
    To reduce code duplication, we introduce a new --dry-run
    option, which can be used by third parties to figure out
    whether calling autogen.sh is necessary or not: this allows
    us to get rid of the reimplementation of part of the logic
    in cfg.mk and guarantee they'll never get out of sync.
    
    Other changes include: making dirty submodules checking and
    cleaning entirely independent of other operations; removing
    the use of 'set -e' and handling errors explicitly instead;
    better parsing of command line arguments.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [libvirt test] 107696: regressions - FAIL
  2017-04-26 20:34 [libvirt test] 107696: regressions - FAIL osstest service owner
@ 2017-05-01 22:58 ` Jim Fehlig
  2017-05-02  9:19   ` Wei Liu
  2017-05-02 10:28   ` Ian Jackson
  0 siblings, 2 replies; 5+ messages in thread
From: Jim Fehlig @ 2017-05-01 22:58 UTC (permalink / raw)
  To: osstest service owner, xen-devel; +Cc: Wei Liu, Ian Jackson

[-- Attachment #1: Type: text/plain, Size: 792 bytes --]

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


[-- Attachment #2: osstest-libvirt-keycodemap-submodule.patch --]
[-- Type: text/x-diff, Size: 907 bytes --]

>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


[-- Attachment #3: Type: text/plain, Size: 127 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [libvirt test] 107696: regressions - FAIL
  2017-05-01 22:58 ` Jim Fehlig
@ 2017-05-02  9:19   ` Wei Liu
  2017-05-02 10:28   ` Ian Jackson
  1 sibling, 0 replies; 5+ messages in thread
From: Wei Liu @ 2017-05-02  9:19 UTC (permalink / raw)
  To: Jim Fehlig; +Cc: Wei Liu, xen-devel, Ian Jackson, osstest service owner

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.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [libvirt test] 107696: regressions - FAIL
  2017-05-01 22:58 ` Jim Fehlig
  2017-05-02  9:19   ` Wei Liu
@ 2017-05-02 10:28   ` Ian Jackson
  2017-05-02 14:33     ` Jim Fehlig
  1 sibling, 1 reply; 5+ messages in thread
From: Ian Jackson @ 2017-05-02 10:28 UTC (permalink / raw)
  To: Jim Fehlig; +Cc: xen-devel, Wei Liu, osstest service owner

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.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [libvirt test] 107696: regressions - FAIL
  2017-05-02 10:28   ` Ian Jackson
@ 2017-05-02 14:33     ` Jim Fehlig
  0 siblings, 0 replies; 5+ messages in thread
From: Jim Fehlig @ 2017-05-02 14:33 UTC (permalink / raw)
  To: Ian Jackson; +Cc: xen-devel, Wei Liu, osstest service owner

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


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-05-02 14:33 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-26 20:34 [libvirt test] 107696: regressions - FAIL osstest service owner
2017-05-01 22:58 ` Jim Fehlig
2017-05-02  9:19   ` Wei Liu
2017-05-02 10:28   ` Ian Jackson
2017-05-02 14:33     ` Jim Fehlig

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.