fix: update resource queries to use max by (container, namespace) for accurate limits and requests

Author: rbjornstad

internal/utilization/queries.go

@@ -14,11 +14,11 @@ import (
 )
 
 const (
-	appCPULimit         = `kube_pod_container_resource_limits{namespace=%q, container=%q, resource="cpu", unit="core"}`
-	appCPURequest       = `kube_pod_container_resource_requests{namespace=%q, container=%q, resource="cpu",unit="core"}`
+	appCPULimit         = `max by (container, namespace) (kube_pod_container_resource_limits{namespace=%q, container=%q, resource="cpu", unit="core"})`
+	appCPURequest       = `max by (container, namespace) (kube_pod_container_resource_requests{namespace=%q, container=%q, resource="cpu",unit="core"})`
 	appCPUUsage         = `rate(container_cpu_usage_seconds_total{namespace=%q, container=%q}[5m])`
-	appMemoryLimit      = `kube_pod_container_resource_limits{namespace=%q, container=%q, resource="memory", unit="byte"}`
-	appMemoryRequest    = `kube_pod_container_resource_requests{namespace=%q, container=%q, resource="memory",unit="byte"}`
+	appMemoryLimit      = `max by (container, namespace) (kube_pod_container_resource_limits{namespace=%q, container=%q, resource="memory", unit="byte"})`
+	appMemoryRequest    = `max by (container, namespace) (kube_pod_container_resource_requests{namespace=%q, container=%q, resource="memory",unit="byte"})`
 	appMemoryUsage      = `last_over_time(container_memory_working_set_bytes{namespace=%q, container=%q}[5m])`
 	instanceCPUUsage    = `rate(container_cpu_usage_seconds_total{namespace=%q, container=%q, pod=%q}[5m])`
 	instanceMemoryUsage = `last_over_time(container_memory_working_set_bytes{namespace=%q, container=%q, pod=%q}[5m])`