Skip to content

fix: map Python CRITICAL log level to OTel FATAL severity text#4365

Merged
xrmx merged 5 commits intoopen-telemetry:mainfrom
Krishnachaitanyakc:fix/map-critical-severity-text-to-fatal
Apr 1, 2026
Merged

fix: map Python CRITICAL log level to OTel FATAL severity text#4365
xrmx merged 5 commits intoopen-telemetry:mainfrom
Krishnachaitanyakc:fix/map-critical-severity-text-to-fatal

Conversation

@Krishnachaitanyakc
Copy link
Copy Markdown
Contributor

Summary

Related: open-telemetry/opentelemetry-python#4984

Test plan

  • Updated test_handler.py assertions from "CRITICAL" to "FATAL"

…trib handler

The OTel specification defines the severity text for the highest severity
range as "FATAL", but the contrib logging handler was passing through
Python's native "CRITICAL" level name unchanged. This mirrors the fix
made in open-telemetry/opentelemetry-python#5011 for the SDK handler.

Related: open-telemetry/opentelemetry-python#4984
@lzchen lzchen moved this to Ready for review in Python PR digest Mar 26, 2026
@Krishnachaitanyakc Krishnachaitanyakc requested a review from a team as a code owner March 26, 2026 17:56
@xrmx xrmx moved this from Ready for review to Approved PRs in Python PR digest Apr 1, 2026
@xrmx xrmx enabled auto-merge (squash) April 1, 2026 15:07
@xrmx xrmx merged commit 6bd19c0 into open-telemetry:main Apr 1, 2026
860 checks passed
@github-project-automation github-project-automation Bot moved this from Approved PRs to Done in Python PR digest Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants