[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] BootLinuxSshTest: Only run the tests when expli
From: |
Cleber Rosa |
Subject: |
Re: [Qemu-devel] [PATCH] BootLinuxSshTest: Only run the tests when explicitly requested |
Date: |
Thu, 19 Sep 2019 09:52:23 -0400 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
On Wed, Sep 18, 2019 at 09:23:59PM -0400, Cleber Rosa wrote:
> On Wed, Sep 18, 2019 at 02:27:48PM +0200, Philippe Mathieu-Daudé wrote:
> > Currently the Avocado framework does not distinct the time spent
> > downloading assets vs. the time spent running a test. With big
> > assets (like a full VM image) the tests likely fail.
> >
> > This is a limitation known by the Avocado team.
> > Until this issue get fixed, do not run this tests automatically.
> >
> > Tests can still be run setting the AVOCADO_TIMEOUT_EXPECTED
> > environment variable.
> >
> > Reported-by: Gerd Hoffmann <address@hidden>
> > Reported-by: David Gibson <address@hidden>
> > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > ---
> > tests/acceptance/linux_ssh_mips_malta.py | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/acceptance/linux_ssh_mips_malta.py
> > b/tests/acceptance/linux_ssh_mips_malta.py
> > index 134f10cac3..7200507a3a 100644
> > --- a/tests/acceptance/linux_ssh_mips_malta.py
> > +++ b/tests/acceptance/linux_ssh_mips_malta.py
> > @@ -12,7 +12,7 @@ import logging
> > import paramiko
> > import time
> >
> > -from avocado import skipIf
> > +from avocado import skipUnless
> > from avocado_qemu import Test
> > from avocado.utils import process
> > from avocado.utils import archive
> > @@ -171,7 +171,7 @@ class LinuxSSH(Test):
> > self.run_common_commands()
> > self.shutdown_via_ssh()
> >
> > - @skipIf(os.getenv('CONTINUOUS_INTEGRATION'), 'Running on Travis-CI')
> > + @skipUnless(os.getenv('AVOCADO_TIMEOUT_EXPECTED'), 'Test might
> > timeout')
>
> This is definitely an improvement, specially because IMO a "make
> check-acceptance" command should be more similar across executions,
> wether on Travis or not. The problem is (which is not new) is that
> users have to resort to the test to learn how to run those tests.
>
> FIY, I'm working on a RFC to have a handful of "make
> check-acceptance-$(TYPE)" targets that hopefully can represent most
> users' needs. The avocado command line would still be available for
> more advanced users.
>
> > def test_mips_malta32eb_kernel3_2_0(self):
> > """
> > :avocado: tags=arch:mips
> > @@ -186,7 +186,7 @@ class LinuxSSH(Test):
> >
> > self.check_mips_malta('be', kernel_path, 'mips')
> >
> > - @skipIf(os.getenv('CONTINUOUS_INTEGRATION'), 'Running on Travis-CI')
> > + @skipUnless(os.getenv('AVOCADO_TIMEOUT_EXPECTED'), 'Test might
> > timeout')
> > def test_mips_malta32el_kernel3_2_0(self):
> > """
> > :avocado: tags=arch:mipsel
> > @@ -201,7 +201,7 @@ class LinuxSSH(Test):
> >
> > self.check_mips_malta('le', kernel_path, 'mips')
> >
> > - @skipIf(os.getenv('CONTINUOUS_INTEGRATION'), 'Running on Travis-CI')
> > + @skipUnless(os.getenv('AVOCADO_TIMEOUT_EXPECTED'), 'Test might
> > timeout')
> > def test_mips_malta64eb_kernel3_2_0(self):
> > """
> > :avocado: tags=arch:mips64
> > @@ -215,7 +215,7 @@ class LinuxSSH(Test):
> > kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
> > self.check_mips_malta('be', kernel_path, 'mips64')
> >
> > - @skipIf(os.getenv('CONTINUOUS_INTEGRATION'), 'Running on Travis-CI')
> > + @skipUnless(os.getenv('AVOCADO_TIMEOUT_EXPECTED'), 'Test might
> > timeout')
> > def test_mips_malta64el_kernel3_2_0(self):
> > """
> > :avocado: tags=arch:mips64el
> > --
> > 2.20.1
> >
>
> Reviewed-by: Cleber Rosa <address@hidden>
BTW, also queueing this one on my python-next branch.