remote: fix wrong event name used for latency (#149973)

This commit is contained in:
Connor Peet 2022-05-19 14:08:15 -07:00 committed by GitHub
parent 92b6c1be54
commit 3b3a3e3e36
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -243,20 +243,20 @@ class InitialRemoteConnectionHealthContribution implements IWorkbenchContributio
await timeout(EXT_HOST_LATENCY_DELAY);
}
type RemoteConnectionFailureClassification = {
type RemoteConnectionLatencyClassification = {
owner: 'connor4312';
comment: 'The latency to the remote extension host';
web: { classification: 'SystemMetaData'; purpose: 'PerformanceAndHealth'; comment: 'Whether this is running on web' };
remoteName: { classification: 'SystemMetaData'; purpose: 'PerformanceAndHealth'; comment: 'Anonymized remote name' };
latencyMs: { classification: 'SystemMetaData'; purpose: 'PerformanceAndHealth'; comment: 'Latency to the remote, in milliseconds'; isMeasurement: true };
};
type RemoteConnectionFailureEvent = {
type RemoteConnectionLatencyEvent = {
web: boolean;
remoteName: string | undefined;
latencyMs: number;
};
this._telemetryService.publicLog2<RemoteConnectionFailureEvent, RemoteConnectionFailureClassification>('remoteConnectionFailure', {
this._telemetryService.publicLog2<RemoteConnectionLatencyEvent, RemoteConnectionLatencyClassification>('remoteConnectionLatency', {
web: isWeb,
remoteName: getRemoteName(this._environmentService.remoteAuthority),
latencyMs: bestLatency