chore: send telemetry on startup (#359)

* chore: send telemetry on startup I noticed that AI lab is doing this - send the extension version in on startup so that we can see which version users have installed. This should also be in the PD telemetry, but easier to track/correlate here. Signed-off-by: Tim deBoer <[email protected]> * chore: remove version Remove version, since Podman Desktop already adds common.extensionVersion. Signed-off-by: Tim deBoer <[email protected]> --------- Signed-off-by: Tim deBoer <[email protected]>
podman-desktop · Apr 22, 2024 · a2ed2c7 · a2ed2c7
1 parent 8121e85
commit a2ed2c7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/packages/backend/src/extension.spec.ts b/packages/backend/src/extension.spec.ts
@@ -26,6 +26,7 @@ import os from 'node:os';
 const originalConsoleLog = console.log;
 
 const mocks = vi.hoisted(() => ({
+  logUsageMock: vi.fn(),
   logErrorMock: vi.fn(),
   consoleLogMock: vi.fn(),
   consoleWarnMock: vi.fn(),
@@ -42,7 +43,7 @@ vi.mock('@podman-desktop/api', async () => {
     version: '1.8.0',
     env: {
       createTelemetryLogger: () => ({
-        logUsage: vi.fn(),
+        logUsage: mocks.logUsageMock,
         logError: mocks.logErrorMock,
       }),
     },
@@ -106,6 +107,7 @@ test('check activate', async () => {
   await activate(fakeContext);
 
   expect(mocks.consoleLogMock).toBeCalledWith('starting bootc extension');
+  expect(mocks.logUsageMock).toHaveBeenCalled();
 });
 
 describe('version checker', () => {

diff --git a/packages/backend/src/extension.ts b/packages/backend/src/extension.ts
@@ -45,6 +45,8 @@ export async function activate(extensionContext: ExtensionContext): Promise<void
     );
   }
 
+  telemetryLogger.logUsage('start');
+
   const history = new History(extensionContext.storagePath);
   await history.loadFile();
-Original file line number
+Diff line change
@@ Expand Up @@
         );
       }
+      telemetryLogger.logUsage('start');
       const history = new History(extensionContext.storagePath);
       await history.loadFile();
@@ Expand Down @@