From 7d8cb3d0e079430f3bf3937d4ded96b06c7c939a Mon Sep 17 00:00:00 2001 From: zounengren Date: Fri, 8 Jul 2022 15:06:03 +0800 Subject: [PATCH] using map instead Signed-off-by: zounengren --- pkg/controller/evpa/container_policy.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/controller/evpa/container_policy.go b/pkg/controller/evpa/container_policy.go index 638073e99..59b1c930c 100644 --- a/pkg/controller/evpa/container_policy.go +++ b/pkg/controller/evpa/container_policy.go @@ -29,6 +29,7 @@ func (c *EffectiveVPAController) ReconcileContainerPolicies(evpa *autoscalingapi rankedEstimators := RankEstimators(resourceEstimators) needReconciledContainers := make(map[string]autoscalingapi.ContainerResourcePolicy) + containerResourceRequirement := make(map[string]*corev1.ResourceRequirements) for _, container := range podTemplate.Spec.Containers { for _, containerPolicy := range evpa.Spec.ResourcePolicy.ContainerPolicies { if containerPolicy.ContainerName == "*" || containerPolicy.ContainerName == container.Name { @@ -37,10 +38,11 @@ func (c *EffectiveVPAController) ReconcileContainerPolicies(evpa *autoscalingapi needReconciledContainers[container.Name] = *out } } + containerResourceRequirement[container.Name] = &container.Resources } for container, containerPolicy := range needReconciledContainers { // get current resource by pod template - resourceRequirement, found := utils.GetResourceByPodTemplate(podTemplate, container) + resourceRequirement, found := containerResourceRequirement[container] if !found { klog.Warningf("ContainerName %s not found", containerPolicy.ContainerName) continue