[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[RFC PATCH 5/9] tests: use tests/venv to run basevm.py-based scripts
From: |
John Snow |
Subject: |
[RFC PATCH 5/9] tests: use tests/venv to run basevm.py-based scripts |
Date: |
Thu, 12 May 2022 20:06:05 -0400 |
This patch co-opts the virtual environment being used by avocado tests
to also run the basevm.py tests. This is being done in preparation for
for the qemu.qmp package being removed from qemu.git.
As part of the change, remove any sys.path() hacks and treat "qemu" as a
normal third-party import.
Signed-off-by: John Snow <jsnow@redhat.com>
---
tests/vm/Makefile.include | 13 +++++++------
tests/vm/basevm.py | 6 +++---
2 files changed, 10 insertions(+), 9 deletions(-)
diff --git a/tests/vm/Makefile.include b/tests/vm/Makefile.include
index ae91f5043e5..588bc999cc9 100644
--- a/tests/vm/Makefile.include
+++ b/tests/vm/Makefile.include
@@ -84,10 +84,11 @@ vm-clean-all:
$(IMAGES_DIR)/%.img: $(SRC_PATH)/tests/vm/% \
$(SRC_PATH)/tests/vm/basevm.py \
- $(SRC_PATH)/tests/vm/Makefile.include
+ $(SRC_PATH)/tests/vm/Makefile.include \
+ check-venv
@mkdir -p $(IMAGES_DIR)
$(call quiet-command, \
- $(PYTHON) $< \
+ $(TESTS_PYTHON) $< \
$(if $(V)$(DEBUG), --debug) \
$(if $(GENISOIMAGE),--genisoimage $(GENISOIMAGE)) \
$(if $(QEMU_LOCAL),--build-path $(BUILD_DIR)) \
@@ -101,9 +102,9 @@ $(IMAGES_DIR)/%.img: $(SRC_PATH)/tests/vm/% \
# Build in VM $(IMAGE)
-vm-build-%: $(IMAGES_DIR)/%.img
+vm-build-%: $(IMAGES_DIR)/%.img check-venv
$(call quiet-command, \
- $(PYTHON) $(SRC_PATH)/tests/vm/$* \
+ $(TESTS_PYTHON) $(SRC_PATH)/tests/vm/$* \
$(if $(V)$(DEBUG), --debug) \
$(if $(DEBUG), --interactive) \
$(if $(J),--jobs $(J)) \
@@ -127,9 +128,9 @@ vm-boot-serial-%: $(IMAGES_DIR)/%.img
-device virtio-net-pci,netdev=vnet \
|| true
-vm-boot-ssh-%: $(IMAGES_DIR)/%.img
+vm-boot-ssh-%: $(IMAGES_DIR)/%.img check-venv
$(call quiet-command, \
- $(PYTHON) $(SRC_PATH)/tests/vm/$* \
+ $(TESTS_PYTHON) $(SRC_PATH)/tests/vm/$* \
$(if $(J),--jobs $(J)) \
$(if $(V)$(DEBUG), --debug) \
$(if $(QEMU_LOCAL),--build-path $(BUILD_DIR)) \
diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py
index 254e11c932b..d7d0413df35 100644
--- a/tests/vm/basevm.py
+++ b/tests/vm/basevm.py
@@ -18,9 +18,6 @@
import logging
import time
import datetime
-sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..', 'python'))
-from qemu.machine import QEMUMachine
-from qemu.utils import get_info_usernet_hostfwd_port, kvm_available
import subprocess
import hashlib
import argparse
@@ -31,6 +28,9 @@
import traceback
import shlex
+from qemu.machine import QEMUMachine
+from qemu.utils import get_info_usernet_hostfwd_port, kvm_available
+
SSH_KEY_FILE = os.path.join(os.path.dirname(__file__),
"..", "keys", "id_rsa")
SSH_PUB_KEY_FILE = os.path.join(os.path.dirname(__file__),
--
2.34.1
- Re: [RFC PATCH 6/9] tests: add check-venv as a dependency of check and check-block, (continued)
[RFC PATCH 4/9] tests: silence pip upgrade warnings during venv creation, John Snow, 2022/05/12
[RFC PATCH 3/9] tests: install "qemu" namespace package into venv, John Snow, 2022/05/12
[RFC PATCH 5/9] tests: use tests/venv to run basevm.py-based scripts,
John Snow <=
[RFC PATCH 8/9] iotests: fix source directory location, John Snow, 2022/05/12
[RFC PATCH 7/9] tests: add check-venv to build-tcg-disabled CI recipe, John Snow, 2022/05/12
[RFC PATCH 9/9] iotests: use tests/venv for running tests, John Snow, 2022/05/12