Skip to content

Commit f052003

Browse files
authored
Merge pull request #5457 from laurazard/backport-dropped-defer
[27.x backport] telemetry: fix early meterprovider shutdown
2 parents e85edf8 + 460f1be commit f052003

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

Diff for: cmd/docker/docker.go

+5-3
Original file line numberDiff line numberDiff line change
@@ -360,9 +360,11 @@ func runDocker(ctx context.Context, dockerCli *command.DockerCli) error {
360360

361361
mp := dockerCli.MeterProvider()
362362
if mp, ok := mp.(command.MeterProvider); ok {
363-
if err := mp.Shutdown(ctx); err != nil {
364-
otel.Handle(err)
365-
}
363+
defer func() {
364+
if err := mp.Shutdown(ctx); err != nil {
365+
otel.Handle(err)
366+
}
367+
}()
366368
} else {
367369
fmt.Fprint(dockerCli.Err(), "Warning: Unexpected OTEL error, metrics may not be flushed")
368370
}

0 commit comments

Comments
 (0)