Uh oh!
There was an error while loading. Please reload this page.
Enhance logging. - #613
Conversation
bcf1b70 to
1627c13Compareyrobla
commented
Jan 17, 2025
I am open to opinion from other colleagues, but i was very used to set log-level=DEBUG to see litellm outpout, so i could see input/output that is sent to LLMs. With this change i cannot see them anymore. So we either improve our debug logs a bit more to include those outputs, or we should be able to set log levels to litellm to the same level as we pass by parameter |
jhrozek
commented
Jan 17, 2025
+1 quite often I need to see what is it that we are sending. I don't know if we can tune litellm debug levels any further, but this is the kind of a request I like to see: I don't care about these: But I'd rather have them than not have any. I wonder if there would be a compromise to set them as another log level (trace or libs or whatnot)? |
lukehinds
commented
Jan 17, 2025
we should be able to bring the others loggers in , i planned on making it so you could configure what other loggers are allowed in, this way you can switch off stuff like asyncio that is really loud when coupled with sqlalchemy I had old branch I never completed work on, where I did something like this to make it configurable: # Configure logger levels for different componentsforlogger_name, levelinlogger_levels.items():
logger=logging.getLogger(logger_name)
logger.setLevel(level)
logger.propagate=True# Default logger levels if not specifiedif"uvicorn"notinlogger_levels:
logging.getLogger("uvicorn").setLevel(log_level.value)
if"sqlalchemy"notinlogger_levels:
logging.getLogger("sqlalchemy").setLevel(log_level.value)
if"asyncio"notinlogger_levels:
logging.getLogger("asyncio").setLevel(log_level.value) |
f75942e to
a499aecCompareThis change adds line numbers to log messages, and synergizes with Copilot specific changes that make exception handling in some of the copilot pipeline more localized, making it easier to track down issues with the proxy. There's no fundamental change in business logic.
a499aec to
230c2fbCompare
This change adds line numbers to log messages, and synergizes with Copilot specific changes that make exception handling in some of the copilot pipeline more localized, making it easier to track down issues with the proxy. There's no fundamental change in business logic.