1
0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00

Correctly handle HorizontalPodAutoscaler metrics with 0 usage

Signed-off-by: Sebastian Malton <sebastian@malton.name>
This commit is contained in:
Sebastian Malton 2022-08-04 12:38:39 -04:00
parent 249927dc80
commit 8542d6cd22

View File

@ -210,8 +210,6 @@ export class HorizontalPodAutoscalerApi extends KubeApi<HorizontalPodAutoscaler>
} }
} }
function getMetricName(metric: HorizontalPodAutoscalerMetricSpec | HorizontalPodAutoscalerMetricStatus): string | undefined { function getMetricName(metric: HorizontalPodAutoscalerMetricSpec | HorizontalPodAutoscalerMetricStatus): string | undefined {
switch (metric.type) { switch (metric.type) {
case HpaMetricType.Resource: case HpaMetricType.Resource:
@ -232,12 +230,12 @@ function getMetricName(metric: HorizontalPodAutoscalerMetricSpec | HorizontalPod
function getResourceMetricValue(currentMetric: ResourceMetricStatus | undefined, targetMetric: ResourceMetricSource): MetricCurrentTarget { function getResourceMetricValue(currentMetric: ResourceMetricStatus | undefined, targetMetric: ResourceMetricSource): MetricCurrentTarget {
return { return {
current: ( current: (
currentMetric?.currentAverageUtilization typeof currentMetric?.currentAverageUtilization === "number"
? `${currentMetric.currentAverageUtilization}%` ? `${currentMetric.currentAverageUtilization}%`
: currentMetric?.currentAverageValue : currentMetric?.currentAverageValue
), ),
target: ( target: (
targetMetric?.targetAverageUtilization typeof targetMetric?.targetAverageUtilization === "number"
? `${targetMetric.targetAverageUtilization}%` ? `${targetMetric.targetAverageUtilization}%`
: targetMetric?.targetAverageValue : targetMetric?.targetAverageValue
), ),
@ -280,12 +278,12 @@ function getExternalMetricValue(currentMetric: ExternalMetricStatus | undefined,
function getContainerResourceMetricValue(currentMetric: ContainerResourceMetricStatus | undefined, targetMetric: ContainerResourceMetricSource): MetricCurrentTarget { function getContainerResourceMetricValue(currentMetric: ContainerResourceMetricStatus | undefined, targetMetric: ContainerResourceMetricSource): MetricCurrentTarget {
return { return {
current: ( current: (
currentMetric?.currentAverageUtilization typeof currentMetric?.currentAverageUtilization === "number"
? `${currentMetric.currentAverageUtilization}%` ? `${currentMetric.currentAverageUtilization}%`
: currentMetric?.currentAverageValue : currentMetric?.currentAverageValue
), ),
target: ( target: (
targetMetric?.targetAverageUtilization typeof targetMetric?.targetAverageUtilization === "number"
? `${targetMetric.targetAverageUtilization}%` ? `${targetMetric.targetAverageUtilization}%`
: targetMetric?.targetAverageValue : targetMetric?.targetAverageValue
), ),