Skip to content

Commit 5be8b80

Browse files
Enrico Usaienrico-usai
authored andcommitted
Split p4d EFA and multi-NICs tests in separate files
EFA tests don't need p4d in the head node and they can be executed with the current code. Instead, to run multi-NICs test it is required to force usage of ODCR for the head node by adding `CapacityReservationSpecification` in the launch template. By splitting the tests is easier to execute them separately. Signed-off-by: Enrico Usai <[email protected]>
1 parent 8070831 commit 5be8b80

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
{%- import 'common.jinja2' as common -%}
2+
{%- set regions = ["us-east-1"] -%}
3+
{%- set instances = ["p4d.24xlarge"] -%}
4+
5+
---
6+
test-suites:
7+
efa:
8+
test_efa.py::test_hit_efa:
9+
dimensions:
10+
- regions: {{ regions }}
11+
instances: {{ instances }}
12+
oss: {{ common.OSS_COMMERCIAL_X86 }}
13+
schedulers: ["slurm"]

tests/integration-tests/configs/p4d.yaml renamed to tests/integration-tests/configs/p4d-multinics.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,6 @@
44

55
---
66
test-suites:
7-
efa:
8-
test_efa.py::test_hit_efa:
9-
dimensions:
10-
- regions: {{ regions }}
11-
instances: {{ instances }}
12-
oss: {{ common.OSS_COMMERCIAL_X86 }}
13-
schedulers: ["slurm"]
147
multiple_nics:
158
test_multiple_nics.py::test_multiple_nics:
169
dimensions:

0 commit comments

Comments
 (0)