Skip to content

Commit 3929fb9

Browse files
authored
fix: analysis with multiple metrics (argoproj#1261)
Signed-off-by: hari rongali <[email protected]>
1 parent 79739fd commit 3929fb9

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

controller/metrics/analysis.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,6 @@ func collectAnalysisTemplate(ch chan<- prometheus.Metric, namespace, name string
106106

107107
for _, metric := range at.Metrics {
108108
metricType := metricproviders.Type(metric)
109-
addGauge(MetricAnalysisTemplateMetricInfo, 1, metricType)
109+
addGauge(MetricAnalysisTemplateMetricInfo, 1, metricType, metric.Name)
110110
}
111111
}

controller/metrics/analysis_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -179,8 +179,8 @@ analysis_template_info{name="http-benchmark-test",namespace=""} 1
179179
analysis_template_info{name="http-benchmark-test",namespace="jesse-test"} 1
180180
# HELP analysis_template_metric_info Information on metrics in analysis templates.
181181
# TYPE analysis_template_metric_info gauge
182-
analysis_template_metric_info{name="http-benchmark-test",namespace="",type="Web"} 1
183-
analysis_template_metric_info{name="http-benchmark-test",namespace="jesse-test",type="Web"} 1
182+
analysis_template_metric_info{metric="webmetric",name="http-benchmark-test",namespace="",type="Web"} 1
183+
analysis_template_metric_info{metric="webmetric",name="http-benchmark-test",namespace="jesse-test",type="Web"} 1
184184
`
185185
registry := prometheus.NewRegistry()
186186
at := newFakeAnalysisTemplate(fakeAnalysisTemplate)

controller/metrics/prommetrics.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ var (
132132
MetricAnalysisTemplateMetricInfo = prometheus.NewDesc(
133133
"analysis_template_metric_info",
134134
"Information on metrics in analysis templates.",
135-
append(namespaceNameLabels, "type"),
135+
append(namespaceNameLabels, "type", "metric"),
136136
nil,
137137
)
138138
)

0 commit comments

Comments
 (0)