SWATCH-2903 Marketplace throws vague error when handling Default api …

…exception Signed-off-by: Kartik Shah <karshah@redhat.com>
RedHatInsights · Oct 1, 2024 · 5792ef7 · 5792ef7
1 parent 4a7fe29
commit 5792ef7
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/...er-aws/src/main/java/com/redhat/swatch/aws/service/AwsBillableUsageAggregateConsumer.java b/...er-aws/src/main/java/com/redhat/swatch/aws/service/AwsBillableUsageAggregateConsumer.java
@@ -130,9 +130,16 @@ public void process(BillableUsageAggregate billableUsageAggregate) {
             billableUsageAggregate.getAggregateKey().getOrgId(),
             billableUsageAggregate.getRemittanceUuids(),
             billableUsageAggregate.getWindowTimestamp(),
-            awsUsageWindow);
+            awsUsageWindow,
+            e);
         emitErrorStatusOnUsage(billableUsageAggregate, BillableUsage.ErrorCode.INACTIVE);
       } else {
+        log.warn(
+            "Subscription not found for for aggregateId={} orgId={} remittanceUUIDs={}",
+            billableUsageAggregate.getAggregateId(),
+            billableUsageAggregate.getAggregateKey().getOrgId(),
+            billableUsageAggregate.getRemittanceUuids(),
+            e);
         emitErrorStatusOnUsage(
             billableUsageAggregate, BillableUsage.ErrorCode.SUBSCRIPTION_NOT_FOUND);
       }

diff --git a/...re/src/main/java/com/redhat/swatch/azure/service/AzureBillableUsageAggregateConsumer.java b/...re/src/main/java/com/redhat/swatch/azure/service/AzureBillableUsageAggregateConsumer.java
@@ -137,9 +137,16 @@ public void process(BillableUsageAggregate billableUsageAggregate) {
             billableUsageAggregate.getAggregateKey().getOrgId(),
             billableUsageAggregate.getRemittanceUuids(),
             billableUsageAggregate.getWindowTimestamp(),
-            azureUsageWindow);
+            azureUsageWindow,
+            e);
         emitErrorStatusOnUsage(billableUsageAggregate, BillableUsage.ErrorCode.INACTIVE);
       } else {
+        log.warn(
+            "Subscription not found for for aggregateId={} orgId={} remittanceUUIDs={}",
+            billableUsageAggregate.getAggregateId(),
+            billableUsageAggregate.getAggregateKey().getOrgId(),
+            billableUsageAggregate.getRemittanceUuids(),
+            e);
         emitErrorStatusOnUsage(
             billableUsageAggregate, BillableUsage.ErrorCode.SUBSCRIPTION_NOT_FOUND);
       }