[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v5 05/20] Acceptance tests: use "arch:" tag to filte
From: |
Cleber Rosa |
Subject: |
[Qemu-devel] [PATCH v5 05/20] Acceptance tests: use "arch:" tag to filter target specific tests |
Date: |
Tue, 12 Mar 2019 13:18:09 -0400 |
Currently, some tests contains target architecture information, in the
form of 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 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>
Reviewed-by: Cornelia Huck <address@hidden>
---
tests/Makefile.include | 3 +++
tests/acceptance/boot_linux_console.py | 2 +-
tests/acceptance/linux_initrd.py | 2 +-
tests/acceptance/virtio_version.py | 2 +-
4 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/tests/Makefile.include b/tests/Makefile.include
index c887f7164d..1ae88db3ed 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -1138,6 +1138,7 @@ TESTS_RESULTS_DIR=$(BUILD_DIR)/tests/results
# Any number of command separated loggers are accepted. For more
# information please refer to "avocado --help".
AVOCADO_SHOW=app
+AVOCADO_TAGS=$(patsubst %-softmmu,-t arch:%, $(filter
%-softmmu,$(TARGET_DIRS)))
ifneq ($(findstring v2,"v$(PYTHON_VERSION)"),v2)
$(TESTS_VENV_DIR): $(TESTS_VENV_REQ)
@@ -1163,6 +1164,8 @@ check-acceptance: check-venv $(TESTS_RESULTS_DIR)
$(call quiet-command, \
$(TESTS_VENV_DIR)/bin/python -m avocado \
--show=$(AVOCADO_SHOW) run --job-results-dir=$(TESTS_RESULTS_DIR) \
+ --filter-by-tags-include-empty --filter-by-tags-include-empty-key \
+ $(AVOCADO_TAGS) \
--failfast=on $(SRC_PATH)/tests/acceptance, \
"AVOCADO", "tests/acceptance")
diff --git a/tests/acceptance/boot_linux_console.py
b/tests/acceptance/boot_linux_console.py
index beeb1e59e8..fa4594f612 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -18,7 +18,7 @@ class BootLinuxConsole(Test):
Boots a x86_64 Linux kernel and checks that the console is operational
and the kernel command line is properly passed from QEMU to the kernel
- :avocado: tags=x86_64
+ :avocado: tags=arch:x86_64
"""
timeout = 60
diff --git a/tests/acceptance/linux_initrd.py b/tests/acceptance/linux_initrd.py
index fbdb48e43f..23be5a63aa 100644
--- a/tests/acceptance/linux_initrd.py
+++ b/tests/acceptance/linux_initrd.py
@@ -19,7 +19,7 @@ class LinuxInitrd(Test):
"""
Checks QEMU evaluates correctly the initrd file passed as -initrd option.
- :avocado: tags=x86_64
+ :avocado: tags=arch:x86_64
"""
timeout = 300
diff --git a/tests/acceptance/virtio_version.py
b/tests/acceptance/virtio_version.py
index 37fc01ea18..8b97453ff8 100644
--- a/tests/acceptance/virtio_version.py
+++ b/tests/acceptance/virtio_version.py
@@ -61,7 +61,7 @@ class VirtioVersionCheck(Test):
same device tree created by `disable-modern` and
`disable-legacy`.
- :avocado: tags=x86_64
+ :avocado: tags=arch:x86_64
"""
# just in case there are failures, show larger diff:
--
2.20.1
- [Qemu-devel] [PATCH v5 00/20] Acceptance Tests: target architecture support, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 01/20] Acceptance tests: show avocado test execution by default, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 02/20] Acceptance tests: improve docstring on pick_default_qemu_bin(), Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 03/20] Acceptance tests: fix doc reference to avocado_qemu directory, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 04/20] Acceptance tests: introduce arch parameter and attribute, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 05/20] Acceptance tests: use "arch:" tag to filter target specific tests,
Cleber Rosa <=
- [Qemu-devel] [PATCH v5 06/20] Acceptance tests: look for target architecture in test tags first, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 08/20] Boot Linux Console Test: update the x86_64 kernel, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 07/20] Boot Linux Console Test: rename the x86_64 after the arch and machine, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 09/20] Boot Linux Console Test: add common kernel command line options, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 10/20] Boot Linux Console Test: increase timeout, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 11/20] Boot Linux Console Test: refactor the console watcher into utility method, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 12/20] scripts/qemu.py: support adding a console with the default serial device, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 13/20] Boot Linux Console Test: add a test for mips + malta, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 14/20] Boot Linux Console Test: add a test for mips64el + malta, Cleber Rosa, 2019/03/12
- [Qemu-devel] [PATCH v5 15/20] Boot Linux Console Test: add a test for ppc64 + pseries, Cleber Rosa, 2019/03/12