Skip to content

Commit eadfaa8

Browse files
prasad0896shvbsle
authored andcommitted
Move neuron and nvidia e2e tests to separate block
1 parent b2521ca commit eadfaa8

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

e2e/setup/e2e.go

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -158,13 +158,19 @@ func TestWrapper(t *testing.T, Testenv env.Environment) {
158158
Testenv.TestInParallel(t,
159159
monitors.ExporterImpl(),
160160
monitors.KernelMonitor(),
161-
monitors.NeuronMonitor(),
162161
monitors.StressFileObserver(),
163162
)
164163
})
165164

166-
// Run nvidia monitor test after all parallel monitors complete.
167-
Testenv.Test(t, monitors.NvidiaMonitor(awsCfg))
165+
// Accelerated hardware monitors run in parallel after the core monitors.
166+
// Neuron and Nvidia are mutually exclusive on a node, so only one will
167+
// run amongst the two on accelerated hardware and will skip on all others.
168+
t.Run("AcceleratedMonitors", func(t *testing.T) {
169+
Testenv.TestInParallel(t,
170+
monitors.NvidiaMonitor(awsCfg),
171+
monitors.NeuronMonitor(),
172+
)
173+
})
168174

169175
// test the addon configuration if the agent is installed as an EKS Addon.
170176
// this is disruptive, so it must run alone.

0 commit comments

Comments
 (0)