[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH 06/18] Acceptance tests: use "arch:" tag to filt
From: |
Cleber Rosa |
Subject: |
Re: [qemu-s390x] [PATCH 06/18] Acceptance tests: use "arch:" tag to filter target specific tests |
Date: |
Wed, 30 Jan 2019 17:15:48 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 1/18/19 5:38 AM, Cornelia Huck wrote:
> On Thu, 17 Jan 2019 13:56:16 -0500
> Cleber Rosa <address@hidden> wrote:
>
>> Currently, the only test that contains some target architecture
>> information is "boot_linux_console.py" which test contains a "x86_64"
>> tag. But that tag is not respected in the default execution, that is,
>> "make check-acceptance" doesn't do anything with it.
>>
>> That said, even the target architecture handling currently present in
>> the "avocado_qemu.Test" class, class is pretty limited. For instance,
>> by default, it chooses a target based on the host architecture.
>>
>> Because the original implementation of the tags feature in Avocado did
>> not include any time of namespace or "key:val" mechanism, no tag has
>> relation to another tag. The new implementation of the tags feature
>> from version 67.0 onwards, allows "key:val" tags, and because of that,
>> a test can be classified with a tag in a given key. For instance, the
>> new proposed version of the "boot_linux_console.py" test, which
>> downloads and attempts to run a x86_64 kernel, is now tagged as:
>>
>> :avocado: tags=arch:x86_64
>>
>> This means that it can be filtered (out) when no x86_64 target is
>> available. At the same time, tests that don't have a "arch:" tag,
>> will not be filtered out.
>>
>> Signed-off-by: Cleber Rosa <address@hidden>
>> ---
>> tests/Makefile.include | 3 +++
>> tests/acceptance/boot_linux_console.py | 2 +-
>> tests/requirements.txt | 2 +-
>> 3 files changed, 5 insertions(+), 2 deletions(-)
>>
>
>> diff --git a/tests/acceptance/boot_linux_console.py
>> b/tests/acceptance/boot_linux_console.py
>> index 98324f7591..46b20bdfe2 100644
>> --- a/tests/acceptance/boot_linux_console.py
>> +++ b/tests/acceptance/boot_linux_console.py
>> @@ -19,7 +19,7 @@ class BootLinuxConsole(Test):
>> and the kernel command line is properly passed from QEMU to the kernel
>>
>> :avocado: enable
>> - :avocado: tags=x86_64
>> + :avocado: tags=arch:x86_64
>> """
>>
>> timeout = 60
>
> You probably want to do the same change in virtio_version.py;
> otherwise, if I run the acceptance tests on s390x, it will run into
> timeouts (it looks like that test is intended to be run with x86
> machines anyway.)
>
Right, good catch. Also, the same applies to the "linux_initrd.py"
test, merged after this version.
- Cleber.
- Re: [qemu-s390x] [PATCH 13/18] Boot Linux Console Test: add a test for mips64el + malta, (continued)
- [qemu-s390x] [PATCH 01/18] scripts/qemu.py: log QEMU launch command line, Cleber Rosa, 2019/01/17
- [qemu-s390x] [PATCH 07/18] Acceptance tests: look for target architecture in test tags first, Cleber Rosa, 2019/01/17
- [qemu-s390x] [PATCH 06/18] Acceptance tests: use "arch:" tag to filter target specific tests, Cleber Rosa, 2019/01/17
- Re: [qemu-s390x] [PATCH 00/18] Acceptance Tests: target architecture support, Aleksandar Markovic, 2019/01/21
- Re: [qemu-s390x] [Qemu-devel] [PATCH 00/18] Acceptance Tests: target architecture support, no-reply, 2019/01/31