[receiver/splunk_hec] Fix parsing of a single metric value payload (o…

…pen-telemetry#33096) Remove redundant `_value` attribute as it's set in the datapoint value. Fixes open-telemetry#33084
shalper2 · May 16, 2024 · ebee48b · ebee48b
1 parent c3b68d6
commit ebee48b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/receiver/splunkhecreceiver/splunkhec_to_metricdata.go b/receiver/splunkhecreceiver/splunkhec_to_metricdata.go
@@ -128,7 +128,7 @@ func buildAttributes(dimensions map[string]any) pcommon.Map {
 	attributes.EnsureCapacity(len(dimensions))
 	for key, val := range dimensions {
 
-		if strings.HasPrefix(key, "metric_name") {
+		if strings.HasPrefix(key, "metric_name") || key == "_value" {
 			continue
 		}
 		if key == "" || val == nil {

diff --git a/receiver/splunkhecreceiver/splunkhec_to_metricdata_test.go b/receiver/splunkhecreceiver/splunkhec_to_metricdata_test.go
@@ -54,6 +54,19 @@ func Test_splunkV2ToMetricsData(t *testing.T) {
 			wantMetricsData: buildDefaultMetricsData(nanos),
 			hecConfig:       defaultTestingHecConfig,
 		},
+		{
+			name: "int_gauge_v7",
+			splunkDataPoint: func() *splunk.Event {
+				pt := buildDefaultSplunkDataPt()
+				delete(pt.Fields, "metric_name:single")
+				pt.Fields["metric_name"] = "single"
+				pt.Fields["_value"] = int64Ptr(13)
+				return pt
+
+			}(),
+			wantMetricsData: buildDefaultMetricsData(nanos),
+			hecConfig:       defaultTestingHecConfig,
+		},
 		{
 			name: "multiple",
 			splunkDataPoint: func() *splunk.Event {