All of lore.kernel.org
 help / color / mirror / Atom feed
* [xen-4.2-testing test] 57630: regressions - FAIL
@ 2015-05-31 22:57 osstest service user
  2015-06-01  7:56 ` Jan Beulich
  0 siblings, 1 reply; 7+ messages in thread
From: osstest service user @ 2015-05-31 22:57 UTC (permalink / raw)
  To: xen-devel; +Cc: ian.jackson

flight 57630 xen-4.2-testing real [real]
http://logs.test-lab.xenproject.org/osstest/logs/57630/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 test-amd64-i386-xend-winxpsp3 16 guest-stop               fail REGR. vs. 53018
 test-amd64-i386-xend-qemut-winxpsp3 16 guest-stop         fail REGR. vs. 53018

Regressions which are regarded as allowable (not blocking):
 test-amd64-i386-xl-qemut-win7-amd64 16 guest-stop              fail like 53018
 test-amd64-amd64-xl-qemut-win7-amd64 16 guest-stop             fail like 53018
 test-amd64-i386-xl-win7-amd64 16 guest-stop                   fail  like 53018

Tests which did not succeed, but are not blocking:
 test-amd64-amd64-rumpuserxen-amd64  1 build-check(1)               blocked n/a
 test-i386-i386-rumpuserxen-i386  1 build-check(1)               blocked  n/a
 test-amd64-i386-rumpuserxen-i386  1 build-check(1)               blocked  n/a
 build-i386-rumpuserxen        5 rumpuserxen-build            fail   never pass
 test-amd64-amd64-xl-qemuu-ovmf-amd64  9 debian-hvm-install     fail never pass
 test-amd64-i386-xl-qemuu-ovmf-amd64  9 debian-hvm-install      fail never pass
 build-amd64-rumpuserxen       5 rumpuserxen-build            fail   never pass
 test-amd64-amd64-libvirt     12 migrate-support-check        fail   never pass
 test-amd64-i386-libvirt      12 migrate-support-check        fail   never pass
 test-i386-i386-libvirt       12 migrate-support-check        fail   never pass
 test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop              fail never pass
 test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop             fail never pass

version targeted for testing:
 xen                  63aeca00c805fa1c47d9f7b1978e83e41ab482d4
baseline version:
 xen                  7e527e2ab6c95ef84035d02e9e50b956a0d469c9

------------------------------------------------------------
People who touched revisions under test:
  Ian Jackson <ian.jackson@eu.citrix.com>
  Petr Matousek <pmatouse@redhat.com>
------------------------------------------------------------

jobs:
 build-amd64                                                  pass    
 build-i386                                                   pass    
 build-amd64-libvirt                                          pass    
 build-i386-libvirt                                           pass    
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 build-amd64-rumpuserxen                                      fail    
 build-i386-rumpuserxen                                       fail    
 test-amd64-amd64-xl                                          pass    
 test-amd64-i386-xl                                           pass    
 test-i386-i386-xl                                            pass    
 test-amd64-i386-rhel6hvm-amd                                 pass    
 test-amd64-i386-qemut-rhel6hvm-amd                           pass    
 test-amd64-i386-qemuu-rhel6hvm-amd                           pass    
 test-amd64-amd64-xl-qemut-debianhvm-amd64                    pass    
 test-amd64-i386-xl-qemut-debianhvm-amd64                     pass    
 test-amd64-amd64-xl-qemuu-debianhvm-amd64                    pass    
 test-amd64-i386-xl-qemuu-debianhvm-amd64                     pass    
 test-amd64-i386-qemuu-freebsd10-amd64                        pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         fail    
 test-amd64-i386-xl-qemuu-ovmf-amd64                          fail    
 test-amd64-amd64-rumpuserxen-amd64                           blocked 
 test-amd64-amd64-xl-qemut-win7-amd64                         fail    
 test-amd64-i386-xl-qemut-win7-amd64                          fail    
 test-amd64-amd64-xl-qemuu-win7-amd64                         fail    
 test-amd64-i386-xl-qemuu-win7-amd64                          fail    
 test-amd64-amd64-xl-win7-amd64                               pass    
 test-amd64-i386-xl-win7-amd64                                fail    
 test-amd64-amd64-xl-credit2                                  pass    
 test-i386-i386-xl-credit2                                    pass    
 test-amd64-i386-qemuu-freebsd10-i386                         pass    
 test-amd64-i386-rumpuserxen-i386                             blocked 
 test-i386-i386-rumpuserxen-i386                              blocked 
 test-amd64-i386-rhel6hvm-intel                               pass    
 test-amd64-i386-qemut-rhel6hvm-intel                         pass    
 test-amd64-i386-qemuu-rhel6hvm-intel                         pass    
 test-amd64-amd64-libvirt                                     pass    
 test-amd64-i386-libvirt                                      pass    
 test-i386-i386-libvirt                                       pass    
 test-amd64-amd64-xl-multivcpu                                pass    
 test-i386-i386-xl-multivcpu                                  pass    
 test-amd64-amd64-pair                                        pass    
 test-amd64-i386-pair                                         pass    
 test-i386-i386-pair                                          pass    
 test-amd64-amd64-xl-sedf-pin                                 pass    
 test-i386-i386-xl-sedf-pin                                   pass    
 test-amd64-amd64-pv                                          pass    
 test-amd64-i386-pv                                           pass    
 test-i386-i386-pv                                            pass    
 test-amd64-amd64-xl-sedf                                     pass    
 test-i386-i386-xl-sedf                                       pass    
 test-amd64-i386-xl-qemut-winxpsp3-vcpus1                     pass    
 test-amd64-i386-xl-qemuu-winxpsp3-vcpus1                     pass    
 test-amd64-i386-xl-winxpsp3-vcpus1                           pass    
 test-amd64-i386-xend-qemut-winxpsp3                          fail    
 test-amd64-amd64-xl-qemut-winxpsp3                           pass    
 test-i386-i386-xl-qemut-winxpsp3                             pass    
 test-amd64-amd64-xl-qemuu-winxpsp3                           pass    
 test-i386-i386-xl-qemuu-winxpsp3                             pass    
 test-amd64-i386-xend-winxpsp3                                fail    
 test-amd64-amd64-xl-winxpsp3                                 pass    
 test-i386-i386-xl-winxpsp3                                   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

Test harness code can be found at
    http://xenbits.xen.org/gitweb?p=osstest.git;a=summary


Not pushing.

------------------------------------------------------------
commit 63aeca00c805fa1c47d9f7b1978e83e41ab482d4
Author: Ian Jackson <ian.jackson@eu.citrix.com>
Date:   Wed May 27 16:57:21 2015 +0100

    QEMU_TAG update
========================================
commit 63a2748f7a97462c9d967daaabaeb5749aee24ea
Author: Petr Matousek <pmatouse@redhat.com>
Date:   Wed May 6 09:48:59 2015 +0200

    fdc: force the fifo access to be in bounds of the allocated buffer
    
    During processing of certain commands such as FD_CMD_READ_ID and
    FD_CMD_DRIVE_SPECIFICATION_COMMAND the fifo memory access could
    get out of bounds leading to memory corruption with values coming
    from the guest.
    
    Fix this by making sure that the index is always bounded by the
    allocated memory.
    
    This is XSA-133 / CVE-2015-3456.
    
    Signed-off-by: Petr Matousek <pmatouse@redhat.com>
    Reviewed-by: John Snow <jsnow@redhat.com>
    (cherry picked from commit 4de1422ea306832b6ef2cba34e9febb73dd139a7)
    (cherry picked from commit e42b84cf1c524d017208b981c7749dc5945dda72)
    (cherry picked from commit c8c6ba08684f2fb8600df479b49c5511d6fed41a)
    (cherry picked from commit c545d0dfcc5ab5e69d0eabc14ec101293236055c)

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

* Re: [xen-4.2-testing test] 57630: regressions - FAIL
  2015-05-31 22:57 [xen-4.2-testing test] 57630: regressions - FAIL osstest service user
@ 2015-06-01  7:56 ` Jan Beulich
  2015-06-01  8:34   ` [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL) Ian Campbell
  0 siblings, 1 reply; 7+ messages in thread
From: Jan Beulich @ 2015-06-01  7:56 UTC (permalink / raw)
  To: xen-devel; +Cc: ian.jackson

>>> On 01.06.15 at 00:57, <osstest@xenbits.xen.org> wrote:
> flight 57630 xen-4.2-testing real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/57630/ 
> 
> Regressions :-(
> 
> Tests which did not succeed and are blocking,
> including tests which could not be run:
>  test-amd64-i386-xend-winxpsp3 16 guest-stop               fail REGR. vs. 53018
>  test-amd64-i386-xend-qemut-winxpsp3 16 guest-stop         fail REGR. vs. 53018

This seems to be due to osstest using an unrecognized option:

2015-05-31 20:32:59 Z executing ssh ... root@172.16.144.32 xm shutdown -wF win.guest.osstest
Error: option -F not recognized
Usage: xm shutdown <Domain> [-waRH]

Shutdown a domain.
[options] [DOM]

Shutdown one or more domains gracefully.

Jan

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

* [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)
  2015-06-01  7:56 ` Jan Beulich
@ 2015-06-01  8:34   ` Ian Campbell
  2015-06-01  9:21     ` Jan Beulich
  2015-06-01 10:02     ` Ian Campbell
  0 siblings, 2 replies; 7+ messages in thread
From: Ian Campbell @ 2015-06-01  8:34 UTC (permalink / raw)
  To: Jan Beulich, ian.jackson; +Cc: xen-devel

On Mon, 2015-06-01 at 08:56 +0100, Jan Beulich wrote:
> >>> On 01.06.15 at 00:57, <osstest@xenbits.xen.org> wrote:
> > flight 57630 xen-4.2-testing real [real]
> > http://logs.test-lab.xenproject.org/osstest/logs/57630/ 
> > 
> > Regressions :-(
> > 
> > Tests which did not succeed and are blocking,
> > including tests which could not be run:
> >  test-amd64-i386-xend-winxpsp3 16 guest-stop               fail REGR. vs. 53018
> >  test-amd64-i386-xend-qemut-winxpsp3 16 guest-stop         fail REGR. vs. 53018
> 
> This seems to be due to osstest using an unrecognized option:
> 
> 2015-05-31 20:32:59 Z executing ssh ... root@172.16.144.32 xm shutdown -wF win.guest.osstest
> Error: option -F not recognized

I think this osstest patch ought to fix it. I've only "tested" with perl
-c though.

--------8>--------


>From aa79bd958fe42da2876fd76e61468c183a7fabdc Mon Sep 17 00:00:00 2001
From: Ian Campbell <ian.campbell@citrix.com>
Date: Mon, 1 Jun 2015 09:32:37 +0100
Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown

This is a feature of xl only.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
 Osstest/Toolstack/xl.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Osstest/Toolstack/xl.pm b/Osstest/Toolstack/xl.pm
index dd12ae1..23328d3 100644
--- a/Osstest/Toolstack/xl.pm
+++ b/Osstest/Toolstack/xl.pm
@@ -56,7 +56,7 @@ sub shutdown_wait ($$$) {
     my $ho = $self->{Host};
     my $gn = $gho->{Name};
     my $acpi_fallback = guest_var($gho,'acpi_shutdown','false') eq 'true'
-	? "F" : "";
+	&& $self->{Name} eq 'xl' ? "F" : "";
     target_cmd_root($ho,"$self->{_Command} shutdown -w${acpi_fallback} $gn", $timeout);
 }
 
-- 
2.1.4

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

* [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)
  2015-06-01  8:34   ` [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL) Ian Campbell
@ 2015-06-01  9:21     ` Jan Beulich
  2015-06-01  9:42       ` Ian Campbell
  2015-06-01 10:02     ` Ian Campbell
  1 sibling, 1 reply; 7+ messages in thread
From: Jan Beulich @ 2015-06-01  9:21 UTC (permalink / raw)
  To: Ian Campbell; +Cc: xen-devel, ian.jackson

>>> On 01.06.15 at 10:34, <ian.campbell@citrix.com> wrote:
> Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown
> 
> This is a feature of xl only.
> 
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> ---
>  Osstest/Toolstack/xl.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Osstest/Toolstack/xl.pm b/Osstest/Toolstack/xl.pm
> index dd12ae1..23328d3 100644
> --- a/Osstest/Toolstack/xl.pm
> +++ b/Osstest/Toolstack/xl.pm
> @@ -56,7 +56,7 @@ sub shutdown_wait ($$$) {
>      my $ho = $self->{Host};
>      my $gn = $gho->{Name};
>      my $acpi_fallback = guest_var($gho,'acpi_shutdown','false') eq 'true'
> -	? "F" : "";
> +	&& $self->{Name} eq 'xl' ? "F" : "";
>      target_cmd_root($ho,"$self->{_Command} shutdown -w${acpi_fallback} $gn", $timeout);

Well - to me, not knowing much about osstest's structure, catering
for xm in a file called xl.pm seems odd. And then, taking into
consideration a hypothetical 3rd toolstack, I wonder whether
assuming only xl supports -F (instead of "only xm does not support
it") is the more suitable check.

Jan

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

* Re: [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)
  2015-06-01  9:21     ` Jan Beulich
@ 2015-06-01  9:42       ` Ian Campbell
  2015-06-01  9:47         ` Jan Beulich
  0 siblings, 1 reply; 7+ messages in thread
From: Ian Campbell @ 2015-06-01  9:42 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel, ian.jackson

On Mon, 2015-06-01 at 10:21 +0100, Jan Beulich wrote:
> >>> On 01.06.15 at 10:34, <ian.campbell@citrix.com> wrote:
> > Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown
> > 
> > This is a feature of xl only.
> > 
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> > ---
> >  Osstest/Toolstack/xl.pm | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/Osstest/Toolstack/xl.pm b/Osstest/Toolstack/xl.pm
> > index dd12ae1..23328d3 100644
> > --- a/Osstest/Toolstack/xl.pm
> > +++ b/Osstest/Toolstack/xl.pm
> > @@ -56,7 +56,7 @@ sub shutdown_wait ($$$) {
> >      my $ho = $self->{Host};
> >      my $gn = $gho->{Name};
> >      my $acpi_fallback = guest_var($gho,'acpi_shutdown','false') eq 'true'
> > -	? "F" : "";
> > +	&& $self->{Name} eq 'xl' ? "F" : "";
> >      target_cmd_root($ho,"$self->{_Command} shutdown -w${acpi_fallback} $gn", $timeout);
> 
> Well - to me, not knowing much about osstest's structure, catering
> for xm in a file called xl.pm seems odd. And then, taking into
> consideration a hypothetical 3rd toolstack, I wonder whether
> assuming only xl supports -F (instead of "only xm does not support
> it") is the more suitable check.

xend.pm inherits from xl.pm in a oo-ish way.

I did consider adding shutdown_wait to xend.pm to overload this one, but
that meant duplicating 95% unaltered.

A third toolstack is not hypothetical, libvirt's virsh is supported, but
virsh.pm sharing no historical legacy with xl/xm inherets from the
toplevel toolstack parent, not the xl one.

Ian.

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

* Re: [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)
  2015-06-01  9:42       ` Ian Campbell
@ 2015-06-01  9:47         ` Jan Beulich
  0 siblings, 0 replies; 7+ messages in thread
From: Jan Beulich @ 2015-06-01  9:47 UTC (permalink / raw)
  To: Ian Campbell; +Cc: xen-devel, ian.jackson

>>> On 01.06.15 at 11:42, <ian.campbell@citrix.com> wrote:
> On Mon, 2015-06-01 at 10:21 +0100, Jan Beulich wrote:
>> >>> On 01.06.15 at 10:34, <ian.campbell@citrix.com> wrote:
>> > Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown
>> > 
>> > This is a feature of xl only.
>> > 
>> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
>> > ---
>> >  Osstest/Toolstack/xl.pm | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> > 
>> > diff --git a/Osstest/Toolstack/xl.pm b/Osstest/Toolstack/xl.pm
>> > index dd12ae1..23328d3 100644
>> > --- a/Osstest/Toolstack/xl.pm
>> > +++ b/Osstest/Toolstack/xl.pm
>> > @@ -56,7 +56,7 @@ sub shutdown_wait ($$$) {
>> >      my $ho = $self->{Host};
>> >      my $gn = $gho->{Name};
>> >      my $acpi_fallback = guest_var($gho,'acpi_shutdown','false') eq 'true'
>> > -	? "F" : "";
>> > +	&& $self->{Name} eq 'xl' ? "F" : "";
>> >      target_cmd_root($ho,"$self->{_Command} shutdown -w${acpi_fallback} 
> $gn", $timeout);
>> 
>> Well - to me, not knowing much about osstest's structure, catering
>> for xm in a file called xl.pm seems odd. And then, taking into
>> consideration a hypothetical 3rd toolstack, I wonder whether
>> assuming only xl supports -F (instead of "only xm does not support
>> it") is the more suitable check.
> 
> xend.pm inherits from xl.pm in a oo-ish way.
> 
> I did consider adding shutdown_wait to xend.pm to overload this one, but
> that meant duplicating 95% unaltered.
> 
> A third toolstack is not hypothetical, libvirt's virsh is supported, but
> virsh.pm sharing no historical legacy with xl/xm inherets from the
> toplevel toolstack parent, not the xl one.

Makes sense, thanks for explaining.

Jan

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

* Re: [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)
  2015-06-01  8:34   ` [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL) Ian Campbell
  2015-06-01  9:21     ` Jan Beulich
@ 2015-06-01 10:02     ` Ian Campbell
  1 sibling, 0 replies; 7+ messages in thread
From: Ian Campbell @ 2015-06-01 10:02 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel, ian.jackson

On Mon, 2015-06-01 at 09:34 +0100, Ian Campbell wrote:
> From aa79bd958fe42da2876fd76e61468c183a7fabdc Mon Sep 17 00:00:00 2001
> From: Ian Campbell <ian.campbell@citrix.com>
> Date: Mon, 1 Jun 2015 09:32:37 +0100
> Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown
> 
> This is a feature of xl only.
> 
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>

With Ian being away for a bit and this blocking testing of older
branches which contain xend I've pushed this to osstest's pretest
branch.

Ian.

> ---
>  Osstest/Toolstack/xl.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Osstest/Toolstack/xl.pm b/Osstest/Toolstack/xl.pm
> index dd12ae1..23328d3 100644
> --- a/Osstest/Toolstack/xl.pm
> +++ b/Osstest/Toolstack/xl.pm
> @@ -56,7 +56,7 @@ sub shutdown_wait ($$$) {
>      my $ho = $self->{Host};
>      my $gn = $gho->{Name};
>      my $acpi_fallback = guest_var($gho,'acpi_shutdown','false') eq 'true'
> -	? "F" : "";
> +	&& $self->{Name} eq 'xl' ? "F" : "";
>      target_cmd_root($ho,"$self->{_Command} shutdown -w${acpi_fallback} $gn", $timeout);
>  }
>  

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

end of thread, other threads:[~2015-06-01 10:02 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-31 22:57 [xen-4.2-testing test] 57630: regressions - FAIL osstest service user
2015-06-01  7:56 ` Jan Beulich
2015-06-01  8:34   ` [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL) Ian Campbell
2015-06-01  9:21     ` Jan Beulich
2015-06-01  9:42       ` Ian Campbell
2015-06-01  9:47         ` Jan Beulich
2015-06-01 10:02     ` Ian Campbell

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.