From 0ee5d43d7450442ad5e5b460fb5a65ad4c3cfbf3 Mon Sep 17 00:00:00 2001 From: vinay kulkarni Date: Thu, 16 Mar 2023 18:35:25 +0000 Subject: [PATCH] Add unit tests covering ephemeral storage resource combinations --- pkg/apis/core/v1/defaults_test.go | 53 +++++++++++++++++++++++++++++++ 1 file changed, 53 insertions(+) diff --git a/pkg/apis/core/v1/defaults_test.go b/pkg/apis/core/v1/defaults_test.go index 4e1979c1907..feeeb12caec 100644 --- a/pkg/apis/core/v1/defaults_test.go +++ b/pkg/apis/core/v1/defaults_test.go @@ -2054,6 +2054,59 @@ func TestSetDefaultResizePolicy(t *testing.T) { }, }, }, + "Ephemeral storage limits are specified": { + testContainer: v1.Container{ + Resources: v1.ResourceRequirements{ + Limits: v1.ResourceList{ + v1.ResourceEphemeralStorage: resource.MustParse("500Mi"), + }, + }, + }, + expectedResizePolicy: nil, + }, + "Ephemeral storage requests and CPU limits are specified": { + testContainer: v1.Container{ + Resources: v1.ResourceRequirements{ + Limits: v1.ResourceList{ + v1.ResourceCPU: resource.MustParse("100m"), + }, + Requests: v1.ResourceList{ + v1.ResourceEphemeralStorage: resource.MustParse("500Mi"), + }, + }, + }, + expectedResizePolicy: []v1.ContainerResizePolicy{ + { + ResourceName: v1.ResourceCPU, + RestartPolicy: v1.NotRequired, + }, + }, + }, + "Ephemeral storage requests and limits, memory requests with restartContainer policy are specified": { + testContainer: v1.Container{ + Resources: v1.ResourceRequirements{ + Limits: v1.ResourceList{ + v1.ResourceEphemeralStorage: resource.MustParse("500Mi"), + }, + Requests: v1.ResourceList{ + v1.ResourceEphemeralStorage: resource.MustParse("500Mi"), + v1.ResourceMemory: resource.MustParse("200Mi"), + }, + }, + ResizePolicy: []v1.ContainerResizePolicy{ + { + ResourceName: v1.ResourceMemory, + RestartPolicy: v1.RestartContainer, + }, + }, + }, + expectedResizePolicy: []v1.ContainerResizePolicy{ + { + ResourceName: v1.ResourceMemory, + RestartPolicy: v1.RestartContainer, + }, + }, + }, } { t.Run(desc, func(t *testing.T) { testPod := v1.Pod{}