[#IOPID-2503] adding unsampled tracking of failures (#247)

Co-authored-by: Greta Quadrati <75862507+gquadrati@users.noreply.github.com>
pagopa · Nov 20, 2024 · 3b55fb3 · 3b55fb3
1 parent 9558674
commit 3b55fb3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/UserDataDeleteOrchestratorV2/handler.ts b/UserDataDeleteOrchestratorV2/handler.ts
@@ -657,10 +657,12 @@ export const createUserDataDeleteOrchestratorHandler = (
         )}`
       );
 
+      // Unsampled tracking of failure
       trackUserDataDeleteException(
         "failed",
         E.toError(error),
-        currentUserDataProcessing
+        currentUserDataProcessing,
+        false
       );
 
       const orchestrationFailure = pipe(
@@ -695,7 +697,8 @@ export const createUserDataDeleteOrchestratorHandler = (
           trackUserDataDeleteException(
             "unhandled_failed_status",
             new Error(readableReport(err)),
-            currentUserDataProcessing
+            currentUserDataProcessing,
+            false
           );
           throw new Error(
             `Activity SetUserDataProcessingStatusActivity (status=FAILED) failed: ${readableReport(

diff --git a/utils/appinsightsEvents.ts b/utils/appinsightsEvents.ts
@@ -24,7 +24,8 @@ export const trackUserDataDeleteEvent = (
 export const trackUserDataDeleteException = (
   eventName: string,
   exception: Error,
-  userDataProcessing: UserDataProcessing
+  userDataProcessing: UserDataProcessing,
+  isSampled: boolean = true
 ) =>
   trackException({
     exception,
@@ -34,7 +35,8 @@ export const trackUserDataDeleteException = (
     },
     tagOverrides: {
       "ai.operation.id": userDataProcessing.userDataProcessingId,
-      "ai.operation.parentId": userDataProcessing.userDataProcessingId
+      "ai.operation.parentId": userDataProcessing.userDataProcessingId,
+      samplingEnabled: String(isSampled)
     }
   });