From 773dcad8a006caab80e10e92bbdf813b85f917dd Mon Sep 17 00:00:00 2001 From: Jiaying Zhang Date: Mon, 9 Oct 2017 12:08:32 -0700 Subject: [PATCH] Extends TestPodContainerDeviceAllocation() to expose issue #53548 that container allocation request may fail if it requests some extended resource not managed by any device plugin. --- pkg/kubelet/cm/device_plugin_handler_test.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/pkg/kubelet/cm/device_plugin_handler_test.go b/pkg/kubelet/cm/device_plugin_handler_test.go index e9df09ef2cc8d..5ae89380d6b1e 100644 --- a/pkg/kubelet/cm/device_plugin_handler_test.go +++ b/pkg/kubelet/cm/device_plugin_handler_test.go @@ -178,6 +178,7 @@ func TestPodContainerDeviceAllocation(t *testing.T) { resourceQuantity2 := *resource.NewQuantity(int64(1), resource.DecimalSI) devId3 := "dev3" devId4 := "dev4" + nonDevicePluginResource := "domain3.com/resource3" m, err := NewDevicePluginManagerTestStub() as := assert.New(t) @@ -216,9 +217,10 @@ func TestPodContainerDeviceAllocation(t *testing.T) { Name: string(uuid.NewUUID()), Resources: v1.ResourceRequirements{ Limits: v1.ResourceList{ - v1.ResourceName(resourceName1): resourceQuantity1, - v1.ResourceName("cpu"): resourceQuantity1, - v1.ResourceName(resourceName2): resourceQuantity2, + v1.ResourceName(resourceName1): resourceQuantity1, + v1.ResourceName("cpu"): resourceQuantity1, + v1.ResourceName(resourceName2): resourceQuantity2, + v1.ResourceName(nonDevicePluginResource): resourceQuantity1, }, }, },