Skip to content

Commit eaf245b

Browse files
committed
Revert "tests/requirements.txt: bump up avocado-framework version to 101.0"
This reverts commit ec5ffa0. Bumping avocado to version 101 has two issues. First, there are problems where Avocado is not logging of command lines or terminal output, and not collecting Python logs outside the avocado namespace. Second, the recent changes to Python handling mean that there is a single virtual environment for all the build, instead of a separate one for testing. Requiring a too-new version of avocado causes conflicts with any avocado plugins installed on the host: $ make check-venv make[1]: Entering directory '/home/berrange/src/virt/qemu/build' GIT ui/keycodemapdb tests/fp/berkeley-testfloat-3 tests/fp/berkeley-softfloat-3 dtc VENVPIP install -e /home/berrange/src/virt/qemu/python/ VENVPIP install -r /home/berrange/src/virt/qemu/tests/requirements.txt ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts. avocado-framework-plugin-varianter-yaml-to-mux 98.0 requires avocado-framework==98.0, but you have avocado-framework 101.0 which is incompatible. avocado-framework-plugin-result-html 98.0 requires avocado-framework==98.0, but you have avocado-framework 101.0 which is incompatible. make[1]: Leaving directory '/home/berrange/src/virt/qemu/build' To avoid this issue, tests/requirements.txt should use a ">=" constraint and the version of Avocado should be limited to what distros provide in the system packages. Only Fedora has Avocado, and more specifically version 92.0 (though 98.0 is also available as a module). As a first step, this patch reverts the introduction of a too-new Avocado. Signed-off-by: Paolo Bonzini <[email protected]>
1 parent 6b40847 commit eaf245b

File tree

2 files changed

+8
-12
lines changed

2 files changed

+8
-12
lines changed

Diff for: tests/Makefile.include

+7-11
Original file line numberDiff line numberDiff line change
@@ -136,18 +136,14 @@ get-vm-image-fedora-31-%: check-venv
136136
# download all vm images, according to defined targets
137137
get-vm-images: check-venv $(patsubst %,get-vm-image-fedora-31-%, $(FEDORA_31_DOWNLOAD))
138138

139-
JOBS_OPTION=$(lastword -j1 $(filter-out -j, $(filter -j%,$(MAKEFLAGS))))
140-
141139
check-avocado: check-venv $(TESTS_RESULTS_DIR) get-vm-images
142-
$(call quiet-command, \
143-
$(PYTHON) -m avocado \
144-
--show=$(AVOCADO_SHOW) run --job-results-dir=$(TESTS_RESULTS_DIR) \
145-
$(if $(AVOCADO_TAGS),, \
146-
--filter-by-tags-include-empty \
147-
--filter-by-tags-include-empty-key) \
148-
--max-parallel-tasks $(JOBS_OPTION:-j%=%) \
149-
$(AVOCADO_CMDLINE_TAGS) \
150-
$(if $(GITLAB_CI),,--failfast) $(AVOCADO_TESTS), \
140+
$(call quiet-command, \
141+
$(PYTHON) -m avocado \
142+
--show=$(AVOCADO_SHOW) run --job-results-dir=$(TESTS_RESULTS_DIR) \
143+
$(if $(AVOCADO_TAGS),, --filter-by-tags-include-empty \
144+
--filter-by-tags-include-empty-key) \
145+
$(AVOCADO_CMDLINE_TAGS) \
146+
$(if $(GITLAB_CI),,--failfast) $(AVOCADO_TESTS), \
151147
"AVOCADO", "tests/avocado")
152148

153149
check-acceptance-deprecated-warning:

Diff for: tests/requirements.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,5 @@
55
# Note that qemu.git/python/ is implicitly installed to this venv when
66
# 'make check-venv' is run, and will persist until configure is run
77
# again.
8-
avocado-framework==101.0
8+
avocado-framework==88.1
99
pycdlib==1.11.0

0 commit comments

Comments
 (0)