ai-workspace-services/litellm
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "fix: set logger levels based on LITELLM_LOG environment variable" (#12122)

Browse Source
This commit is contained in:
Ishaan Jaff 2025-06-27 17:55:44 -07:00 committed by GitHub
parent ebf6395bc1
commit 01b9e6efb1
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 0 additions and 68 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
litellm/_logging.py
View File

@ -103,11 +103,6 @@ verbose_proxy_logger = logging.getLogger("LiteLLM Proxy")
verbose_router_logger = logging.getLogger("LiteLLM Router")
verbose_logger = logging.getLogger("LiteLLM")
# Set the logger level based on LITELLM_LOG environment variable
verbose_proxy_logger.setLevel(numeric_level)
verbose_router_logger.setLevel(numeric_level)
verbose_logger.setLevel(numeric_level)
# Add the handler to the logger
verbose_router_logger.addHandler(handler)
verbose_proxy_logger.addHandler(handler)

63
tests/test_litellm/test_logger_level_initialization.py
View File

@ -1,63 +0,0 @@
"""Test that loggers respect LITELLM_LOG environment variable"""
import logging
import os
import sys
import pytest
sys.path.insert(0, os.path.abspath("../../.."))
def test_logger_level_respects_env_var(monkeypatch):
"""
Test that loggers are initialized with the correct level based on LITELLM_LOG env var.
This verifies the fix for issue #9815 where loggers ignored the LITELLM_LOG setting.
"""
# Test different log levels
test_cases = [
("DEBUG", logging.DEBUG),
("INFO", logging.INFO),
("WARNING", logging.WARNING),
("ERROR", logging.ERROR),
("CRITICAL", logging.CRITICAL),
]
for env_level, expected_numeric_level in test_cases:
# Set the environment variable
monkeypatch.setenv("LITELLM_LOG", env_level)
# Re-import the logging module to pick up the new env var
import importlib
import litellm._logging
importlib.reload(litellm._logging)
# Check that all loggers have the correct level
assert litellm._logging.verbose_logger.level == expected_numeric_level, \
f"verbose_logger level should be {expected_numeric_level} for LITELLM_LOG={env_level}"
assert litellm._logging.verbose_proxy_logger.level == expected_numeric_level, \
f"verbose_proxy_logger level should be {expected_numeric_level} for LITELLM_LOG={env_level}"
assert litellm._logging.verbose_router_logger.level == expected_numeric_level, \
f"verbose_router_logger level should be {expected_numeric_level} for LITELLM_LOG={env_level}"
def test_handler_and_logger_levels_match(monkeypatch):
"""
Test that both handler and logger levels are set to the same value.
This ensures consistent behavior across the logging system.
"""
monkeypatch.setenv("LITELLM_LOG", "WARNING")
# Re-import to apply settings
import importlib
import litellm._logging
importlib.reload(litellm._logging)
# Get the handler level (first handler should be our configured one)
handler_level = litellm._logging.handler.level
# All loggers should have the same level as the handler
assert litellm._logging.verbose_logger.level == handler_level
assert litellm._logging.verbose_proxy_logger.level == handler_level
assert litellm._logging.verbose_router_logger.level == handler_level