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

[Cleber Rosa]

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 <address@hidden> 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.