From ad22b74c60940d0c2332884e37c8996ccbb0a165 Mon Sep 17 00:00:00 2001 From: Patrick Ohly Date: Mon, 9 Sep 2024 18:11:34 +0200 Subject: [PATCH] DRA scheduler: fix match attribute names in test FullyQualifiedNames must include a domain. The current code doesn't care, but once it does, the tests better should behave correctly. --- .../structured/allocator_test.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/staging/src/k8s.io/dynamic-resource-allocation/structured/allocator_test.go b/staging/src/k8s.io/dynamic-resource-allocation/structured/allocator_test.go index f16c38c9cee..d90ad29db6f 100644 --- a/staging/src/k8s.io/dynamic-resource-allocation/structured/allocator_test.go +++ b/staging/src/k8s.io/dynamic-resource-allocation/structured/allocator_test.go @@ -353,11 +353,11 @@ func sliceWithOneDevice(name string, nodeSelection any, pool, driver string) *re } func TestAllocator(t *testing.T) { - nonExistentAttribute := resourceapi.FullyQualifiedName("NonExistentAttribute") - boolAttribute := resourceapi.FullyQualifiedName("boolAttribute") - stringAttribute := resourceapi.FullyQualifiedName("stringAttribute") - versionAttribute := resourceapi.FullyQualifiedName("driverVersion") - intAttribute := resourceapi.FullyQualifiedName("numa") + nonExistentAttribute := resourceapi.FullyQualifiedName(driverA + "/" + "NonExistentAttribute") + boolAttribute := resourceapi.FullyQualifiedName(driverA + "/" + "boolAttribute") + stringAttribute := resourceapi.FullyQualifiedName(driverA + "/" + "stringAttribute") + versionAttribute := resourceapi.FullyQualifiedName(driverA + "/" + "driverVersion") + intAttribute := resourceapi.FullyQualifiedName(driverA + "/" + "numa") testcases := map[string]struct { adminAccess bool