fix: Exception thrown when metric is not found (#1546)

* fix: Exception thrown when metric is not found Signed-off-by: Tom Kerkhove <[email protected]> * Revert Signed-off-by: Tom Kerkhove <[email protected]> * Update changelog Signed-off-by: Tom Kerkhove <[email protected]> * Align expectation with Azure subscription Signed-off-by: Tom Kerkhove <[email protected]>
tomkerkhove · Mar 3, 2021 · 0f59ea8 · 0f59ea8
1 parent 24a52f4
commit 0f59ea8
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/changelog/content/released/v2.1.1-scraper.md b/changelog/content/released/v2.1.1-scraper.md
@@ -0,0 +1,10 @@
+---
+subtitle: "released on 2021-03-03"
+date: 2021-03-03T10:38:47+02:00
+weight: 211-scraper
+version: Scraper - v2.1.1
+---
+
+- {{% tag fixed %}} Issue when metric (values) where not found causing exceptions to be thrown ([#1545](https://github.com/tomkerkhove/promitor/issues/1545))
+
+Full release notes can be found [here](https://github.com/tomkerkhove/promitor/releases/tag/2.1.1-scraper).
diff --git a/src/Promitor.Integrations.AzureMonitor/AzureMonitorClient.cs b/src/Promitor.Integrations.AzureMonitor/AzureMonitorClient.cs
@@ -82,7 +82,7 @@ public async Task<List<MeasuredMetric>> QueryMetricAsync(string metricName, stri
             var relevantMetric = await GetRelevantMetric(metricName, aggregationType, closestAggregationInterval, metricFilter, metricDimension, metricDefinition, startQueryingTime);
             if (relevantMetric.Timeseries.Count < 1)
             {
-                throw new MetricInformationNotFoundException(metricName,metricDimension, "No time series was found");
+                throw new MetricInformationNotFoundException(metricName,"No time series was found", metricDimension);
             }
 
             var measuredMetrics = new List<MeasuredMetric>();

diff --git a/src/Promitor.Tests.Integration/Services/ResourceDiscovery/ResourceDiscoveryTests.cs b/src/Promitor.Tests.Integration/Services/ResourceDiscovery/ResourceDiscoveryTests.cs
@@ -138,7 +138,7 @@ public async Task ResourceDiscovery_GetWithFilterOnOneSubscription_ReturnsExpect
         {
             // Arrange
             const string resourceDiscoveryGroupName = "one-subscriptions-scenario";
-            const int expectedResourceCount = 2;
+            const int expectedResourceCount = 1;
             var resourceDiscoveryClient = new ResourceDiscoveryClient(Configuration, Logger);
 
             // Act