Skip to content

Commit 29ed1fb

Browse files
authored
Merge pull request #132399 from carlory/fix-132397
Fix [Failing test] [sig-node] [Feature:GPUDevicePlugin] [Serial]-related tests
2 parents 7521de1 + 5e54df3 commit 29ed1fb

File tree

1 file changed

+18
-4
lines changed

1 file changed

+18
-4
lines changed

test/e2e/node/gpu.go

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -293,17 +293,31 @@ func SetupEnvironmentAndSkipIfNeeded(ctx context.Context, f *framework.Framework
293293
}
294294
}
295295

296+
func isControlPlaneNode(node v1.Node) bool {
297+
_, isControlPlane := node.Labels["node-role.kubernetes.io/control-plane"]
298+
if isControlPlane {
299+
framework.Logf("Node: %q is a control-plane node (label)", node.Name)
300+
return true
301+
}
302+
303+
for _, taint := range node.Spec.Taints {
304+
if taint.Key == "node-role.kubernetes.io/control-plane" {
305+
framework.Logf("Node: %q is a control-plane node (taint)", node.Name)
306+
return true
307+
}
308+
}
309+
framework.Logf("Node: %q is NOT a control-plane node", node.Name)
310+
return false
311+
}
312+
296313
func areGPUsAvailableOnAllSchedulableNodes(ctx context.Context, clientSet clientset.Interface) error {
297314
framework.Logf("Getting list of Nodes from API server")
298315
nodeList, err := clientSet.CoreV1().Nodes().List(ctx, metav1.ListOptions{})
299316
if err != nil {
300317
return fmt.Errorf("unexpected error getting node list: %w", err)
301318
}
302319
for _, node := range nodeList.Items {
303-
if node.Spec.Unschedulable {
304-
continue
305-
}
306-
if _, ok := node.Labels[framework.ControlPlaneLabel]; ok {
320+
if node.Spec.Unschedulable || isControlPlaneNode(node) {
307321
continue
308322
}
309323
framework.Logf("gpuResourceName %s", e2egpu.NVIDIAGPUResourceName)

0 commit comments

Comments
 (0)