fix(Telemetry): Correctly report outcome for interactive setup

serverless · Jul 6, 2021 · 0c5b8dd · 0c5b8dd
1 parent 502f7e7
commit 0c5b8dd
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/scripts/serverless.js b/scripts/serverless.js
@@ -72,7 +72,7 @@ process.on('SIGINT', () => {
       serverless,
       commandUsage,
     });
-    storeTelemetryLocally({ ...telemetryPayload, outcome: 'interrupted' });
+    storeTelemetryLocally({ ...telemetryPayload, outcome: 'interrupt' });
   }
   process.exit(1);
 });
@@ -445,16 +445,17 @@ const processSpanPromise = (async () => {
 
       hasTelemetryBeenReported = true;
       if (!isTelemetryDisabled) {
-        storeTelemetryLocally(
-          generateTelemetryPayload({
+        storeTelemetryLocally({
+          ...generateTelemetryPayload({
             command,
             options,
             commandSchema,
             serviceDir,
             configuration: configurationFromInteractive,
             commandUsage,
-          })
-        );
+          }),
+          outcome: 'success',
+        });
         await sendTelemetry({ serverlessExecutionSpan: processSpanPromise });
       }
       return;