-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[Bug]: LiteLLM logs certain error messages to stdout instead of stderr #12580
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
colesmcintosh
wants to merge
8
commits into
BerriAI:main
Choose a base branch
from
colesmcintosh:fix/stderr-logging-12525
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[Bug]: LiteLLM logs certain error messages to stdout instead of stderr #12580
colesmcintosh
wants to merge
8
commits into
BerriAI:main
from
colesmcintosh:fix/stderr-logging-12525
+84
−17
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Modified exception_mapping_utils.py to print debug info to stderr - Modified get_llm_provider_logic.py to print provider info to stderr - Added tests to verify stderr output behavior Fixes BerriAI#12525
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…nts in exception mapping and LLM provider logic. Update tests to verify logging behavior.
…ic files to clean up code.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Title
Fix error messages being logged to stdout instead of stderr
Relevant issues
Fixes #12525
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/
directory, Adding at least 1 test is a hard requirement - see detailsmake test-unit
Type
🐛 Bug Fix
Changes
Summary
Fixed print statements that were incorrectly outputting error/debug messages to stdout instead of stderr. This was causing issues for applications that expect structured output on stdout.
What was changed:
Modified
litellm/litellm_core_utils/exception_mapping_utils.py
:sys
import at the top of the filefile=sys.stderr
parameterModified
litellm/litellm_core_utils/get_llm_provider_logic.py
:sys
import at the top of the filefile=sys.stderr
parameterAdded
tests/test_litellm/test_exception_mapping_stderr.py
:test_exception_debug_info_goes_to_stderr
: Verifies that debug info is written to stderr, not stdouttest_exception_debug_info_suppressed
: Verifies that suppression still works correctlyTest Results:
Impact:
This fix ensures that error/debug messages from LiteLLM don't interfere with applications that expect structured output on stdout, particularly Python scripts that rely on a one-to-one correspondence between stdin input and stdout output.