fix: Global Default Region and Account not set in certain metrics

Adds them to the metric in adaptMetric if they are present in globalDefault
cdklabs · Dec 18, 2024 · 5aa9986 · 5aa9986
1 parent 54d6a06
commit 5aa9986
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 15 deletions.
diff --git a/lib/common/metric/MetricFactory.ts b/lib/common/metric/MetricFactory.ts
@@ -229,6 +229,14 @@ export class MetricFactory {
    * @param metric metric to be adapted
    */
   adaptMetric(metric: MetricWithAlarmSupport): MetricWithAlarmSupport {
+    if (this.globalDefaults.region) {
+      metric = metric.with({ region: this.globalDefaults.region });
+    }
+
+    if (this.globalDefaults.account) {
+      metric = metric.with({ account: this.globalDefaults.account });
+    }
+
     return metric.with({
       period: this.globalDefaults.period ?? DefaultMetricPeriod,
     });