test: Remove test_max_cpus.py

test_max_cpus.py tests vCPUs are online only on a microVM without CPU templates, which is now covered as part of test_all_vcpus_online(). Remove the test and use the maximum number of vCPUs in the new tests. Signed-off-by: Takahiro Itazuri <itazur@amazon.com>
firecracker-microvm · Dec 9, 2024 · 9705961 · 9705961
1 parent a600777
commit 9705961
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 23 deletions.
diff --git a/...ion_tests/functional/test_cpu_multiple.py → ...egration_tests/functional/test_cpu_all.py b/...ion_tests/functional/test_cpu_multiple.py → ...egration_tests/functional/test_cpu_all.py
@@ -9,7 +9,13 @@
 are operating identically, except for the expected differences.
 """
 
+import pytest
 
+# Use the maximum number of vCPUs supported by Firecracker
+MAX_VCPUS = 32
+
+
+@pytest.mark.parametrize("vcpu_count", [MAX_VCPUS])
 def test_all_vcpus_online(uvm_any):
     """Check all vCPUs are online inside guest"""
     vm = uvm_any
@@ -22,6 +28,7 @@ def test_all_vcpus_online(uvm_any):
         )
 
 
+@pytest.mark.parametrize("vcpu_count", [MAX_VCPUS])
 def test_all_vcpus_have_same_features(uvm_any):
     """
     Check all vCPUs have the same features inside guest.

diff --git a/tests/integration_tests/functional/test_max_vcpus.py b/tests/integration_tests/functional/test_max_vcpus.py