Do not set container.requests for v1

pull/6/head
derekwaynecarr 2015-05-27 14:27:01 -04:00
parent 0ede6dc421
commit 05f099211f
2 changed files with 2 additions and 9 deletions

View File

@ -145,10 +145,6 @@ func defaultContainerResourceRequirements(limitRange *api.LimitRange) api.Resour
value := v.Copy()
requirements.Limits[k] = *value
}
for k, v := range limit.Min {
value := v.Copy()
requirements.Requests[k] = *value
}
}
}
return requirements

View File

@ -84,7 +84,7 @@ func TestDefaultContainerResourceRequirements(t *testing.T) {
limitRange := validLimitRange()
expected := api.ResourceRequirements{
Limits: getResourceList("50m", "5Mi"),
Requests: getResourceList("25m", "1Mi"),
Requests: api.ResourceList{},
}
actual := defaultContainerResourceRequirements(&limitRange)
@ -118,10 +118,7 @@ func TestMergePodResourceRequirements(t *testing.T) {
api.ResourceCPU: defaultRequirements.Limits[api.ResourceCPU],
api.ResourceMemory: resource.MustParse("512Mi"),
},
Requests: api.ResourceList{
api.ResourceCPU: defaultRequirements.Requests[api.ResourceCPU],
api.ResourceMemory: defaultRequirements.Requests[api.ResourceMemory],
},
Requests: api.ResourceList{},
}
mergePodResourceRequirements(&pod, &defaultRequirements)
for i := range pod.Spec.Containers {