Skip to content

Commit eece675

Browse files
committed
fix typo and add unit test for float target metric
1 parent 2f5b0f0 commit eece675

File tree

2 files changed

+17
-3
lines changed

2 files changed

+17
-3
lines changed

internal/store/horizontalpodautoscaler.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ func hpaMetricFamilies(allowAnnotationsList, allowLabelsList []string) []generat
175175
continue
176176
}
177177

178-
for i := range v {
178+
for i := range ok {
179179
if ok[i] {
180180
ms = append(ms, &metric.Metric{
181181
LabelKeys: targetMetricLabels,

internal/store/horizontalpodautoscaler_test.go

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,19 @@ func TestHPAStore(t *testing.T) {
8080
},
8181
Target: autoscaling.MetricTarget{
8282
Value: resourcePtr(resource.MustParse("10")),
83-
AverageValue: resourcePtr(resource.MustParse("0.5")),
83+
AverageValue: resourcePtr(resource.MustParse("12")),
84+
},
85+
},
86+
},
87+
{
88+
Type: autoscaling.ObjectMetricSourceType,
89+
Object: &autoscaling.ObjectMetricSource{
90+
Metric: autoscaling.MetricIdentifier{
91+
Name: "connections",
92+
},
93+
Target: autoscaling.MetricTarget{
94+
Value: resourcePtr(resource.MustParse("0.5")),
95+
AverageValue: resourcePtr(resource.MustParse("0.7")),
8496
},
8597
},
8698
},
@@ -193,8 +205,10 @@ func TestHPAStore(t *testing.T) {
193205
kube_horizontalpodautoscaler_spec_min_replicas{horizontalpodautoscaler="hpa1",namespace="ns1"} 2
194206
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="cpu",metric_target_type="utilization",namespace="ns1"} 80
195207
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="events",metric_target_type="average",namespace="ns1"} 30
196-
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="hits",metric_target_type="average",namespace="ns1"} 0.5
208+
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="hits",metric_target_type="average",namespace="ns1"} 12
197209
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="hits",metric_target_type="value",namespace="ns1"} 10
210+
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="connections",metric_target_type="average",namespace="ns1"} 0.7
211+
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="connections",metric_target_type="value",namespace="ns1"} 0.5
198212
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="memory",metric_target_type="average",namespace="ns1"} 819200
199213
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="memory",metric_target_type="utilization",namespace="ns1"} 80
200214
kube_horizontalpodautoscaler_spec_target_metric{horizontalpodautoscaler="hpa1",metric_name="sqs_jobs",metric_target_type="value",namespace="ns1"} 30

0 commit comments

Comments
 (0)