[Metricbeat][Azure]Check for metrics dimensions before matching two metrics together (#42591) (#42620)

* Check for metrics dimensions before matching two metrics together

* Add test case for metrics with different dimensions

(cherry picked from commit d9f5498)

Co-authored-by: Michalis Katsoulis <[email protected]>

Author: mergify[bot]

x-pack/metricbeat/module/azure/client_utils.go

@@ -92,7 +92,7 @@ func metricIsEmpty(metric armmonitor.MetricValue) bool {
 // matchMetrics will compare current metrics
 func matchMetrics(prevMet Metric, met Metric) bool {
 	if prevMet.Namespace == met.Namespace && reflect.DeepEqual(prevMet.Names, met.Names) && prevMet.ResourceId == met.ResourceId &&
-		prevMet.Aggregations == met.Aggregations && prevMet.TimeGrain == met.TimeGrain {
+		prevMet.Aggregations == met.Aggregations && prevMet.TimeGrain == met.TimeGrain && reflect.DeepEqual(prevMet.Dimensions, met.Dimensions) {
 		return true
 	}
 	return false

x-pack/metricbeat/module/azure/client_utils_test.go

@@ -77,11 +77,17 @@ func TestMatchMetrics(t *testing.T) {
 		Values:       []MetricValue{},
 		TimeGrain:    "1PM",
 	}
+	// match
 	result := matchMetrics(prev, current)
 	assert.True(t, result)
+	// different resourceId, not match
 	current.ResourceId = "id1"
 	result = matchMetrics(prev, current)
 	assert.False(t, result)
+	// different dimension, not match
+	current.Dimensions = []Dimension{{Name: "location", Value: "East Europe"}}
+	result = matchMetrics(prev, current)
+	assert.False(t, result)
 }
 
 func TestMetricIsEmpty(t *testing.T) {