Skip to content

Commit 61fd745

Browse files
authored
[POA-2766] Send accurate duration in initial and success telemetry (#70)
A follow-up PR from #65 (comment) NOTE: Do not merge before postman-eng/observability-superstar-service#2256 is released all the way through to PRODUCTION
1 parent 7ed99da commit 61fd745

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

apidump/apidump.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ func (a *apidump) SendInitialTelemetry() {
214214
req := kgxapi.PostInitialClientTelemetryRequest{
215215
ClientID: a.ClientID,
216216
ObservedStartingAt: a.startTime,
217-
ObservedDurationInSeconds: a.TelemetryInterval,
217+
ObservedDurationInSeconds: 0,
218218
SendsWitnessPayloads: a.ReproMode,
219219
CLIVersion: version.ReleaseVersion().String(),
220220
CLITargetArch: architecture.GetCanonicalArch(),
@@ -438,7 +438,8 @@ func (a *apidump) TelemetryWorker(done <-chan struct{}) {
438438
subsequentTelemetrySent = true
439439
case <-a.successTelemetry.Channel:
440440
if !subsequentTelemetrySent {
441-
a.SendPacketTelemetry(a.TelemetryInterval)
441+
duration := int(time.Since(a.startTime) / time.Second)
442+
a.SendPacketTelemetry(duration)
442443
}
443444
}
444445
}

0 commit comments

Comments
 (0)