Skip to content

Commit f51238e

Browse files
committed
Add limit-tests to jobs.py and debian_uefi_vm as a new marker
Signed-off-by: Andrii Sultanov <[email protected]>
1 parent 3f057f7 commit f51238e

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

jobs.py

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -462,6 +462,19 @@
462462
"nb_pools": 2,
463463
"params": {},
464464
"paths": ["tests/misc/test_pool.py"],
465+
},
466+
"limit-tests": {
467+
"description": "Tests verifying we can hit our supported limits",
468+
"requirements": [
469+
"1 XCP-ng host >= 8.2"
470+
],
471+
"nb_pools": 1,
472+
"params": {
473+
# The test does not work on Alpine because of how it handles
474+
# multiple interfaces on the same network, so use Debian instead
475+
"--vm": "single/debian_uefi_vm",
476+
},
477+
"paths": ["tests/limits/test_vif_limit.py"],
465478
}
466479
}
467480

@@ -653,7 +666,7 @@ def extract_tests(cmd):
653666

654667
print("*** Checking that all tests that use VMs have VM target markers (small_vm, etc.)... ", end="")
655668
tests_missing_vm_markers = extract_tests(
656-
["pytest", "--collect-only", "-q", "-m", "not no_vm and not (small_vm or multi_vm or big_vm)"]
669+
["pytest", "--collect-only", "-q", "-m", "not no_vm and not (small_vm or multi_vm or big_vm or debian_uefi_vm)"]
657670
)
658671
if tests_missing_vm_markers:
659672
error = True

pytest.ini

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ markers =
3030
small_vm: tests that it is enough to run just once, using the smallest possible VM.
3131
big_vm: tests that it would be good to run with a big VM.
3232
multi_vms: tests that it would be good to run on a variety of VMs (includes `small_vm` but excludes `big_vm`).
33+
debian_uefi_vm: tests that require a Debian UEFI VM
3334

3435
# * Other markers
3536
reboot: tests that reboot one or more hosts.

0 commit comments

Comments
 (0)