[awsemfexporter] Fix grouping for container insights metrics for mixed metric types

exporter/awsemfexporter/grouped_metric.go

@@ -91,6 +91,15 @@
 				// them together into one EMF log event, so don't set batchIndex when it's a summary metric
 				metadata.batchIndex = i
 			}
+
+			// Handle metric types with container insights
+			if metadata.receiver == containerInsightsReceiver {
+				// For container insights, treat gauge metrics as sum
+				if metadata.groupedMetricMetadata.metricDataType == pmetric.MetricTypeGauge {
+					metadata.groupedMetricMetadata.metricDataType = pmetric.MetricTypeSum
+				}
+			}
+
 			groupKey := aws.NewKey(metadata.groupedMetricMetadata, labels)
 			if _, ok := groupedMetrics[groupKey]; ok {
 				// if MetricName already exists in metrics map, print warning log

exporter/awsemfexporter/metric_translator.go

@@ -146,7 +146,8 @@ func (mt metricTranslator) translateOTelToGroupedMetric(rm pmetric.ResourceMetri
 		if strings.HasPrefix(serviceName.Str(), "containerInsightsKubeAPIServerScraper") ||
 			strings.HasPrefix(serviceName.Str(), "containerInsightsDCGMExporterScraper") ||
 			strings.HasPrefix(serviceName.Str(), "containerInsightsNeuronMonitorScraper") ||
-			strings.HasPrefix(serviceName.Str(), "containerInsightsKueueMetricsScraper") {
+			strings.HasPrefix(serviceName.Str(), "containerInsightsKueueMetricsScraper") ||
+			strings.HasPrefix(serviceName.Str(), "containerInsightsNVMeExporterScraper") {
 			// the prometheus metrics that come from the container insight receiver need to be clearly tagged as coming from container insights
 			metricReceiver = containerInsightsReceiver
 		}

exporter/awsemfexporter/metric_translator_test.go

@@ -278,6 +278,8 @@
 	neuronMetric.Resource().Attributes().PutStr(conventions.AttributeServiceName, "containerInsightsNeuronMonitorScraper")
 	kueueMetric := createTestResourceMetricsHelper(defaultNumberOfTestMetrics + 1)
 	kueueMetric.Resource().Attributes().PutStr(conventions.AttributeServiceName, "containerInsightsKueueMetricsScraper")
+	nvmeMetric := createTestResourceMetricsHelper(defaultNumberOfTestMetrics + 1)
+	nvmeMetric.Resource().Attributes().PutStr(conventions.AttributeServiceName, "containerInsightsNVMeExporterScraper")
 
 	counterSumMetrics := map[string]*metricInfo{
 		"spanCounter": {
@@ -395,6 +397,32 @@
 			"myServiceNS/containerInsightsKueueMetricsScraper",
 			containerInsightsReceiver,
 		},
+		{
+			"nvme receiver",
+			nvmeMetric,
+			map[string]string{
+				"isItAnError": "false",
+				"spanName":    "testSpan",
+			},
+			map[string]string{
+				"spanName": "testSpan",
+			},
+			"myServiceNS/containerInsightsNVMeExporterScraper",
+			containerInsightsReceiver,
+		},
+		{
+			"container insights gauge/sum metrics",
+			containerInsightMetric,
+			map[string]string{
+				"isItAnError": "false",
+				"spanName":    "testSpan",
+			},
+			map[string]string{
+				"spanName": "testSpan",
+			},
+			"myServiceNS/containerInsightsKubeAPIServerScraper",
+			containerInsightsReceiver,
+		},
 	}
 
 	for _, tc := range testCases {
@@ -416,6 +444,15 @@
 					}
 				}
 
+
+				if tc.expectedReceiver == containerInsightsReceiver {
+					assert.Equal(t, pmetric.MetricTypeSum, v.metadata.metricDataType)
+					assert.True(t, reflect.DeepEqual(counterSumMetrics, v.metrics) ||
+						reflect.DeepEqual(counterGaugeMetrics, v.metrics),
+					)
+					continue
+				}
+
 				switch v.metadata.metricDataType {
 				case pmetric.MetricTypeSum:
 					assert.Len(t, v.metrics, 2)