Skip to content

Make OpenTelemetry provider shutdown idempotent - #35642

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/f3c8769146ee57102477eb9ee7fd948f688efb79
Jul 27, 2026
Merged

Make OpenTelemetry provider shutdown idempotent#35642
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/f3c8769146ee57102477eb9ee7fd948f688efb79

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 27, 2026

Copy link
Copy Markdown

Make OpenTelemetry provider shutdown idempotent

What changed

  • Guard OtelProvider::shutdown so exporters shut down at most once, including when an explicit shutdown is followed by Drop.
  • Let tracer provider shutdown handle flushing without a separate force_flush call.
  • Add regression tests for explicit shutdown and drop-only shutdown.

## What changed

- Guard `OtelProvider::shutdown` so exporters shut down at most once, including when an explicit shutdown is followed by `Drop`.
- Let tracer provider shutdown handle flushing without a separate `force_flush` call.
- Add regression tests for explicit shutdown and drop-only shutdown.

GitOrigin-RevId: f3c8769146ee57102477eb9ee7fd948f688efb79
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/f3c8769146ee57102477eb9ee7fd948f688efb79 branch from 54dfdf9 to 0affaf2 Compare July 27, 2026 16:35
@copyberry
copyberry Bot merged commit 0affaf2 into main Jul 27, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/f3c8769146ee57102477eb9ee7fd948f688efb79 branch July 27, 2026 16:36
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants