github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #117171 from brianpursley/kubectl-1110-tests

Browse Source 
Add unit tests showing the effect of unlimited containers when calculating pod limits
This commit is contained in:
Kubernetes Prow Robot
2023-04-11 21:17:35 -07:00
committed by GitHub
parent 743cc9db44 b04ca79445
commit f5401624ee
1 changed files with 61 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

61
pkg/api/v1/resource/helpers_test.go
View File

@@ -984,6 +984,67 @@ func TestPodResourceLimits(t *testing.T) {
}, },
}, },
}, },
{
description: "no limited containers should result in no limits for the pod",
expectedLimits: v1.ResourceList{},
initContainers: []v1.Container{},
containers: []v1.Container{
{
// Unlimited container
},
},
},
{
description: "one limited and one unlimited container should result in the limited container's limits for the pod",
expectedLimits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("2"),
v1.ResourceMemory: resource.MustParse("2Gi"),
},
initContainers: []v1.Container{},
containers: []v1.Container{
{
Resources: v1.ResourceRequirements{
Limits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("2"),
v1.ResourceMemory: resource.MustParse("2Gi"),
},
},
},
{
// Unlimited container
},
},
},
{
description: "one limited and one unlimited init container should result in the limited init container's limits for the pod",
expectedLimits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("2"),
v1.ResourceMemory: resource.MustParse("2Gi"),
},
initContainers: []v1.Container{
{
Resources: v1.ResourceRequirements{
Limits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("2"),
v1.ResourceMemory: resource.MustParse("2Gi"),
},
},
},
{
// Unlimited init container
},
},
containers: []v1.Container{
{
Resources: v1.ResourceRequirements{
Limits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("1"),
v1.ResourceMemory: resource.MustParse("1Gi"),
},
},
},
},
},
} }
for _, tc := range testCases { for _, tc := range testCases {
p := &v1.Pod{ p := &v1.Pod{