From 10e0b16e42851bf3631b4c2016280c96ef92d3bb Mon Sep 17 00:00:00 2001 From: "grafana-delivery-bot[bot]" <132647405+grafana-delivery-bot[bot]@users.noreply.github.com> Date: Wed, 23 Oct 2024 14:49:01 +0300 Subject: [PATCH] [v11.0.x] Azure: Fix duplicated traces in multi-resource trace query (#95244) Azure: Fix duplicated traces in multi-resource trace query (#95156) Use first resource as base resource for query (cherry picked from commit 8bb7475e4f3bcd790ae4cb5278681a99f87760bf) Co-authored-by: Andreas Christou --- .../azuremonitor/loganalytics/azure-log-analytics-datasource.go | 2 +- .../loganalytics/azure-log-analytics-datasource_test.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource.go b/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource.go index 6fb1fdac45d..ed0b67b2f1a 100644 --- a/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource.go +++ b/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource.go @@ -493,7 +493,7 @@ func (e *AzureLogAnalyticsDatasource) createRequest(ctx context.Context, queryUR } if query.AppInsightsQuery { - body["applications"] = query.Resources + body["applications"] = []string{query.Resources[0]} } jsonValue, err := json.Marshal(body) diff --git a/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource_test.go b/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource_test.go index cfddd704cf1..d21be5bbca6 100644 --- a/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource_test.go +++ b/pkg/tsdb/azuremonitor/loganalytics/azure-log-analytics-datasource_test.go @@ -1622,7 +1622,7 @@ func TestLogAnalyticsCreateRequest(t *testing.T) { TimeColumn: "timestamp", }) require.NoError(t, err) - expectedBody := fmt.Sprintf(`{"applications":["/subscriptions/test-sub/resourceGroups/test-rg/providers/Microsoft.Insights/components/r1","/subscriptions/test-sub/resourceGroups/test-rg/providers/Microsoft.Insights/components/r2"],"query":"","query_datetimescope_column":"timestamp","query_datetimescope_from":"%s","query_datetimescope_to":"%s","timespan":"%s/%s"}`, from.Format(time.RFC3339), to.Format(time.RFC3339), from.Format(time.RFC3339), to.Format(time.RFC3339)) + expectedBody := fmt.Sprintf(`{"applications":["/subscriptions/test-sub/resourceGroups/test-rg/providers/Microsoft.Insights/components/r1"],"query":"","query_datetimescope_column":"timestamp","query_datetimescope_from":"%s","query_datetimescope_to":"%s","timespan":"%s/%s"}`, from.Format(time.RFC3339), to.Format(time.RFC3339), from.Format(time.RFC3339), to.Format(time.RFC3339)) body, err := io.ReadAll(req.Body) require.NoError(t, err) if !cmp.Equal(string(body), expectedBody) {