[Qemu-devel] [PULL 0/6] Python queue, 2019-06-11

Eduardo Habkost posted 6 patches 4 years, 9 months ago
Test s390x passed
Test checkpatch passed
Test asan passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test FreeBSD passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190611171456.23444-1-ehabkost@redhat.com
Maintainers: Fam Zheng <fam@euphon.net>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Aleksandar Rikalo <arikalo@wavecomp.com>, Aleksandar Markovic <amarkovic@wavecomp.com>, Aurelien Jarno <aurelien@aurel32.net>
.travis.yml                            |   2 +-
tests/acceptance/boot_linux_console.py | 145 ++++++++++++++++++++++++-
2 files changed, 142 insertions(+), 5 deletions(-)
[Qemu-devel] [PULL 0/6] Python queue, 2019-06-11
Posted by Eduardo Habkost 4 years, 9 months ago
Changes from 2019-06-07: Python 2 deprecation and Python 3 check
for 3.5+ were removed.

The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:

  Merge remote-tracking branch 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-06-11 16:02:07 +0100)

are available in the Git repository at:

  git://github.com/ehabkost/qemu.git tags/python-next-pull-request

for you to fetch changes up to 6d7a134da4afebe8551a69329478415cfb4cbe91:

  travis: Make check-acceptance job more verbose (2019-06-11 14:13:09 -0300)

----------------------------------------------------------------
Python queue, 2019-06-11

* New boot_linux_console test cases (Philippe Mathieu-Daudé)
* Make check-acceptance Travis job more verbose (Eduardo Habkost)

----------------------------------------------------------------

Eduardo Habkost (1):
  travis: Make check-acceptance job more verbose

Philippe Mathieu-Daudé (5):
  tests/boot_linux_console: Let extract_from_deb handle various
    compressions
  BootLinuxConsoleTest: Do not log empty lines
  BootLinuxConsoleTest: Test the SmartFusion2 board
  BootLinuxConsoleTest: Test nanoMIPS kernels on the I7200 CPU
  BootLinuxConsoleTest: Run kerneltests BusyBox on Malta

 .travis.yml                            |   2 +-
 tests/acceptance/boot_linux_console.py | 145 ++++++++++++++++++++++++-
 2 files changed, 142 insertions(+), 5 deletions(-)

-- 
2.18.0.rc1.1.g3f1ff2140


Re: [Qemu-devel] [PULL 0/6] Python queue, 2019-06-11
Posted by Peter Maydell 4 years, 9 months ago
On Tue, 11 Jun 2019 at 18:15, Eduardo Habkost <ehabkost@redhat.com> wrote:
>
> Changes from 2019-06-07: Python 2 deprecation and Python 3 check
> for 3.5+ were removed.
>
> The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:
>
>   Merge remote-tracking branch 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-06-11 16:02:07 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/ehabkost/qemu.git tags/python-next-pull-request
>
> for you to fetch changes up to 6d7a134da4afebe8551a69329478415cfb4cbe91:
>
>   travis: Make check-acceptance job more verbose (2019-06-11 14:13:09 -0300)
>
> ----------------------------------------------------------------
> Python queue, 2019-06-11
>
> * New boot_linux_console test cases (Philippe Mathieu-Daudé)
> * Make check-acceptance Travis job more verbose (Eduardo Habkost)
>
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/4.1
for any user-visible changes.

-- PMM

Re: [Qemu-devel] [PULL 0/6] Python queue, 2019-06-11
Posted by Peter Maydell 4 years, 9 months ago
On Tue, 11 Jun 2019 at 18:15, Eduardo Habkost <ehabkost@redhat.com> wrote:
>
> Changes from 2019-06-07: Python 2 deprecation and Python 3 check
> for 3.5+ were removed.
>
> The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:
>
>   Merge remote-tracking branch 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-06-11 16:02:07 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/ehabkost/qemu.git tags/python-next-pull-request
>
> for you to fetch changes up to 6d7a134da4afebe8551a69329478415cfb4cbe91:
>
>   travis: Make check-acceptance job more verbose (2019-06-11 14:13:09 -0300)
>
> ----------------------------------------------------------------
> Python queue, 2019-06-11
>
> * New boot_linux_console test cases (Philippe Mathieu-Daudé)
> * Make check-acceptance Travis job more verbose (Eduardo Habkost)
>
> ----------------------------------------------------------------
>
> Eduardo Habkost (1):
>   travis: Make check-acceptance job more verbose
>
> Philippe Mathieu-Daudé (5):
>   tests/boot_linux_console: Let extract_from_deb handle various
>     compressions
>   BootLinuxConsoleTest: Do not log empty lines
>   BootLinuxConsoleTest: Test the SmartFusion2 board
>   BootLinuxConsoleTest: Test nanoMIPS kernels on the I7200 CPU
>   BootLinuxConsoleTest: Run kerneltests BusyBox on Malta

Hi -- it looks like the Travis check-acceptance run started
failing when this pull was merged. Could somebody have a look
at it, please?

https://travis-ci.org/qemu/qemu/jobs/544724911

thanks
-- PMM

Re: [Qemu-devel] [PULL 0/6] Python queue, 2019-06-11
Posted by Cleber Rosa 4 years, 9 months ago
On Tue, Jun 18, 2019 at 02:27:29PM +0100, Peter Maydell wrote:
> On Tue, 11 Jun 2019 at 18:15, Eduardo Habkost <ehabkost@redhat.com> wrote:
> >
> > Changes from 2019-06-07: Python 2 deprecation and Python 3 check
> > for 3.5+ were removed.
> >
> > The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:
> >
> >   Merge remote-tracking branch 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-06-11 16:02:07 +0100)
> >
> > are available in the Git repository at:
> >
> >   git://github.com/ehabkost/qemu.git tags/python-next-pull-request
> >
> > for you to fetch changes up to 6d7a134da4afebe8551a69329478415cfb4cbe91:
> >
> >   travis: Make check-acceptance job more verbose (2019-06-11 14:13:09 -0300)
> >
> > ----------------------------------------------------------------
> > Python queue, 2019-06-11
> >
> > * New boot_linux_console test cases (Philippe Mathieu-Daudé)
> > * Make check-acceptance Travis job more verbose (Eduardo Habkost)
> >
> > ----------------------------------------------------------------
> >
> > Eduardo Habkost (1):
> >   travis: Make check-acceptance job more verbose
> >
> > Philippe Mathieu-Daudé (5):
> >   tests/boot_linux_console: Let extract_from_deb handle various
> >     compressions
> >   BootLinuxConsoleTest: Do not log empty lines
> >   BootLinuxConsoleTest: Test the SmartFusion2 board
> >   BootLinuxConsoleTest: Test nanoMIPS kernels on the I7200 CPU
> >   BootLinuxConsoleTest: Run kerneltests BusyBox on Malta
> 
> Hi -- it looks like the Travis check-acceptance run started
> failing when this pull was merged. Could somebody have a look
> at it, please?
> 
> https://travis-ci.org/qemu/qemu/jobs/544724911
>
> thanks
> -- PMM

Hi Peter,

I'm looking at it.  Strangely enough, I don't see any error or
failures, but only a couple of skips (besides passes):

 PASS 01-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_x86_64_pc
 PASS 02-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_mips_malta
 PASS 03-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_mips64el_malta
 PASS 04-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_mips_malta_cpio
 PASS 05-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_aarch64_virt
 PASS 06-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_arm_virt
 PASS 07-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_arm_emcraft_sf2
 PASS 08-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_s390x_s390_ccw_virtio
 PASS 09-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_alpha_clipper
 PASS 10-/home/travis/build/qemu/qemu/tests/acceptance/cpu_queries.py:QueryCPUModelExpansion.test
 PASS 11-/home/travis/build/qemu/qemu/tests/acceptance/empty_cpu_model.py:EmptyCPUModel.test
 PASS 12-/home/travis/build/qemu/qemu/tests/acceptance/linux_initrd.py:LinuxInitrd.test_with_2gib_file_should_exit_error_msg_with_linux_v3_6
 PASS 13-/home/travis/build/qemu/qemu/tests/acceptance/linux_initrd.py:LinuxInitrd.test_with_2gib_file_should_work_with_linux_v4_16
 SKIP 14-/home/travis/build/qemu/qemu/tests/acceptance/linux_ssh_mips_malta.py:LinuxSSH.test_mips_malta32eb_kernel3_2_0 -> TestSkipError: Running on Travis-CI
 SKIP 15-/home/travis/build/qemu/qemu/tests/acceptance/linux_ssh_mips_malta.py:LinuxSSH.test_mips_malta64el_kernel3_2_0 -> TestSkipError: Running on Travis-CI
 PASS 16-/home/travis/build/qemu/qemu/tests/acceptance/migration.py:Migration.test_migration_with_tcp_localhost
 PASS 17-/home/travis/build/qemu/qemu/tests/acceptance/version.py:Version.test_qmp_human_info_version
 PASS 18-/home/travis/build/qemu/qemu/tests/acceptance/virtio_version.py:VirtioVersionCheck.test_conventional_devs
 PASS 19-/home/travis/build/qemu/qemu/tests/acceptance/virtio_version.py:VirtioVersionCheck.test_modern_only_devs
 PASS 20-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_no_vnc
 PASS 21-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_no_vnc_change_password
 PASS 22-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_vnc_change_password_requires_a_password
 PASS 23-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_vnc_change_password

And Avocado *should* return 0 for those results (I tried it on my
system and this is indeed what I get).  I'm investigating other
possibilities and will let you know.

Regards,
- Cleber.

Re: [Qemu-devel] [PULL 0/6] Python queue, 2019-06-11
Posted by Cleber Rosa 4 years, 9 months ago
On Tue, Jun 18, 2019 at 10:01:53AM -0400, Cleber Rosa wrote:
> On Tue, Jun 18, 2019 at 02:27:29PM +0100, Peter Maydell wrote:
> > On Tue, 11 Jun 2019 at 18:15, Eduardo Habkost <ehabkost@redhat.com> wrote:
> > >
> > > Changes from 2019-06-07: Python 2 deprecation and Python 3 check
> > > for 3.5+ were removed.
> > >
> > > The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:
> > >
> > >   Merge remote-tracking branch 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-06-11 16:02:07 +0100)
> > >
> > > are available in the Git repository at:
> > >
> > >   git://github.com/ehabkost/qemu.git tags/python-next-pull-request
> > >
> > > for you to fetch changes up to 6d7a134da4afebe8551a69329478415cfb4cbe91:
> > >
> > >   travis: Make check-acceptance job more verbose (2019-06-11 14:13:09 -0300)
> > >
> > > ----------------------------------------------------------------
> > > Python queue, 2019-06-11
> > >
> > > * New boot_linux_console test cases (Philippe Mathieu-Daudé)
> > > * Make check-acceptance Travis job more verbose (Eduardo Habkost)
> > >
> > > ----------------------------------------------------------------
> > >
> > > Eduardo Habkost (1):
> > >   travis: Make check-acceptance job more verbose
> > >
> > > Philippe Mathieu-Daudé (5):
> > >   tests/boot_linux_console: Let extract_from_deb handle various
> > >     compressions
> > >   BootLinuxConsoleTest: Do not log empty lines
> > >   BootLinuxConsoleTest: Test the SmartFusion2 board
> > >   BootLinuxConsoleTest: Test nanoMIPS kernels on the I7200 CPU
> > >   BootLinuxConsoleTest: Run kerneltests BusyBox on Malta
> > 
> > Hi -- it looks like the Travis check-acceptance run started
> > failing when this pull was merged. Could somebody have a look
> > at it, please?
> > 
> > https://travis-ci.org/qemu/qemu/jobs/544724911
> >
> > thanks
> > -- PMM
> 
> Hi Peter,
> 
> I'm looking at it.  Strangely enough, I don't see any error or
> failures, but only a couple of skips (besides passes):
> 
>  PASS 01-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_x86_64_pc
>  PASS 02-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_mips_malta
>  PASS 03-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_mips64el_malta
>  PASS 04-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_mips_malta_cpio
>  PASS 05-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_aarch64_virt
>  PASS 06-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_arm_virt
>  PASS 07-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_arm_emcraft_sf2
>  PASS 08-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_s390x_s390_ccw_virtio
>  PASS 09-/home/travis/build/qemu/qemu/tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_alpha_clipper
>  PASS 10-/home/travis/build/qemu/qemu/tests/acceptance/cpu_queries.py:QueryCPUModelExpansion.test
>  PASS 11-/home/travis/build/qemu/qemu/tests/acceptance/empty_cpu_model.py:EmptyCPUModel.test
>  PASS 12-/home/travis/build/qemu/qemu/tests/acceptance/linux_initrd.py:LinuxInitrd.test_with_2gib_file_should_exit_error_msg_with_linux_v3_6
>  PASS 13-/home/travis/build/qemu/qemu/tests/acceptance/linux_initrd.py:LinuxInitrd.test_with_2gib_file_should_work_with_linux_v4_16
>  SKIP 14-/home/travis/build/qemu/qemu/tests/acceptance/linux_ssh_mips_malta.py:LinuxSSH.test_mips_malta32eb_kernel3_2_0 -> TestSkipError: Running on Travis-CI
>  SKIP 15-/home/travis/build/qemu/qemu/tests/acceptance/linux_ssh_mips_malta.py:LinuxSSH.test_mips_malta64el_kernel3_2_0 -> TestSkipError: Running on Travis-CI
>  PASS 16-/home/travis/build/qemu/qemu/tests/acceptance/migration.py:Migration.test_migration_with_tcp_localhost
>  PASS 17-/home/travis/build/qemu/qemu/tests/acceptance/version.py:Version.test_qmp_human_info_version
>  PASS 18-/home/travis/build/qemu/qemu/tests/acceptance/virtio_version.py:VirtioVersionCheck.test_conventional_devs
>  PASS 19-/home/travis/build/qemu/qemu/tests/acceptance/virtio_version.py:VirtioVersionCheck.test_modern_only_devs
>  PASS 20-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_no_vnc
>  PASS 21-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_no_vnc_change_password
>  PASS 22-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_vnc_change_password_requires_a_password
>  PASS 23-/home/travis/build/qemu/qemu/tests/acceptance/vnc.py:Vnc.test_vnc_change_password
> 
> And Avocado *should* return 0 for those results (I tried it on my
> system and this is indeed what I get).  I'm investigating other
> possibilities and will let you know.
>
> Regards,
> - Cleber.

I take that back. The simple "egrep" I didn't show the interruption
during test tearDown() that ocurred here for aarch64:

 - https://travis-ci.org/qemu/qemu/jobs/544724911#L3909

And here for arm:

 - https://travis-ci.org/qemu/qemu/jobs/544724911#L3959

With the revert that Eduardo just proposed, it should be eaiser to see
that those tests actually finished with status INTERRUPTED.

Now, the core issue Which is this related to this:

 - https://bugs.launchpad.net/qemu/+bug/1829779

Some people contributed to discussions around this, and the general
outcome is that we need to workaround the current sitation in the
test code.  So, I'm working on a patch and will send ASAP.

- Cleber.