Skip to content

Conversation

@qianl15
Copy link
Member

@qianl15 qianl15 commented Dec 1, 2025

This PR fixes the issue where DBOS would throw an exception if enable_otlp is set to True but the logger/tracer providers weren't set. Instead of crashing the entire app, it'll log a warning after this change.

Also fixed an issue where DBOS.trace being None (when OTLP tracer wasn't set) could crash the FastAPI app. It should only set the status code if it can find an active span.

@dosubot
Copy link

dosubot bot commented Dec 1, 2025

Documentation Updates

1 document(s) were updated by changes in this PR:

DBOS Python

How did I do? Any feedback?  Join Discord

@qianl15 qianl15 merged commit d05e364 into main Dec 2, 2025
19 of 20 checks passed
@qianl15 qianl15 deleted the qian/otel-logger-warning branch December 2, 2025 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants