Fix prometheus name duplicate _total suffix

Signed-off-by: Bing Han <[email protected]>

CHANGELOG.md

@@ -40,6 +40,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 - The `go.opentelemetry.io/otel/exporters/prometheus` exporter now correctly adds `_total` suffixes to counter metrics. (#3360)
 - The `go.opentelemetry.io/otel/exporters/prometheus` exporter now adds a unit suffix to metric names.
    This can be disabled using the `WithoutUnits()` option added to that package. (#3352)
+- The `go.opentelemetry.io/otel/exporters/prometheus` exporter fixes duplicated `_total` suffixes. (#3368)
 
 ## [1.11.0/0.32.3] 2022-10-12
 

exporters/prometheus/exporter.go

@@ -203,11 +203,11 @@ func getSumMetricData[N int64 | float64](sum metricdata.Sum[N], m metricdata.Met
 		valueType = prometheus.GaugeValue
 	}
 	dataPoints := make([]*metricData, 0, len(sum.DataPoints))
+	if sum.IsMonotonic {
+		// Add _total suffix for counters
+		name += counterSuffix
+	}
 	for _, dp := range sum.DataPoints {
-		if sum.IsMonotonic {
-			// Add _total suffix for counters
-			name += counterSuffix
-		}
 		keys, values := getAttrs(dp.Attributes)
 		desc := prometheus.NewDesc(name, m.Description, keys, nil)
 		md := &metricData{

exporters/prometheus/exporter_test.go

@@ -62,6 +62,14 @@ func TestPrometheusExporter(t *testing.T) {
 				counter.Add(ctx, 5, attrs...)
 				counter.Add(ctx, 10.3, attrs...)
 				counter.Add(ctx, 9, attrs...)
+
+				attrs2 := []attribute.KeyValue{
+					attribute.Key("A").String("D"),
+					attribute.Key("C").String("B"),
+					attribute.Key("E").Bool(true),
+					attribute.Key("F").Int(42),
+				}
+				counter.Add(ctx, 5, attrs2...)
 			},
 		},
 		{

exporters/prometheus/testdata/counter.txt

@@ -1,6 +1,7 @@
 # HELP foo_milliseconds_total a simple counter
 # TYPE foo_milliseconds_total counter
 foo_milliseconds_total{A="B",C="D",E="true",F="42"} 24.3
+foo_milliseconds_total{A="D",C="B",E="true",F="42"} 5
 # HELP target_info Target metadata
 # TYPE target_info gauge
 target_info{service_name="prometheus_test",telemetry_sdk_language="go",telemetry_sdk_name="opentelemetry",telemetry_sdk_version="latest"} 1