fixed typo and tested

Author: dhruvigajjar

src/shared/telemetry/telemetryClient.ts

@@ -78,8 +78,6 @@ export class DefaultTelemetryClient implements TelemetryClient {
      * @param batch batch of events
      */
     public async postMetrics(batch: MetricDatum[]): Promise<MetricDatum[] | undefined> {
-        this.logger.info('Trying to print compute')
-        this.logger.info(getComputeEnvType())
         try {
             // If our batching logic rejected all of the telemetry, don't try to post
             if (batch.length === 0) {
@@ -93,15 +91,13 @@ export class DefaultTelemetryClient implements TelemetryClient {
                         AWSProductVersion: extensionVersion,
                         ClientID: this.clientId,
                         OS: os.platform(),
-                        OSVersion: 'myOsVersion',
+                        OSVersion: os.release(),
                         ComputeEnv: getComputeEnvType(),
                         ParentProduct: vscode.env.appName,
                         ParentProductVersion: vscode.version,
                         MetricData: batch,
                     })
                     .promise()
-                // this.logger.info(client)
-                this.logger.info(getComputeEnvType())
                 this.logger.info(`telemetry: sent batch (size=${batch.length})`)
             } else {
                 this.logger.info(`telemetry: (test mode) dropped batch (size=${batch.length})`)