Merge pull request #115485 from SergeyKanzhelev/epmeneralContainersQoS

small comment the ephemeral containers are not accounted for the QoS …
This commit is contained in:
Kubernetes Prow Robot
2023-02-03 08:46:29 -08:00
committed by GitHub
2 changed files with 2 additions and 0 deletions

View File

@@ -40,6 +40,7 @@ func GetPodQOS(pod *core.Pod) core.PodQOSClass {
limits := core.ResourceList{}
zeroQuantity := resource.MustParse("0")
isGuaranteed := true
// note, ephemeral containers are not considered for QoS as they cannot define resources
allContainers := []core.Container{}
allContainers = append(allContainers, pod.Spec.Containers...)
allContainers = append(allContainers, pod.Spec.InitContainers...)

View File

@@ -37,6 +37,7 @@ func GetPodQOS(pod *core.Pod) core.PodQOSClass {
limits := core.ResourceList{}
zeroQuantity := resource.MustParse("0")
isGuaranteed := true
// note, ephemeral containers are not considered for QoS as they cannot define resources
allContainers := []core.Container{}
allContainers = append(allContainers, pod.Spec.Containers...)
allContainers = append(allContainers, pod.Spec.InitContainers...)