Unverified Commit ebda5196 authored by Russell Bryant's avatar Russell Bryant Committed by GitHub
Browse files

[Core] Fix broken log configuration (#10458)


Signed-off-by: default avatarRussell Bryant <rbryant@redhat.com>
parent 9195dbdb
......@@ -118,7 +118,7 @@ configuration for the root vLLM logger and for the logger you wish to silence:
{
"formatters": {
"vllm": {
"class": "vllm.logging.NewLineFormatter",
"class": "vllm.logging_utils.NewLineFormatter",
"datefmt": "%m-%d %H:%M:%S",
"format": "%(levelname)s %(asctime)s %(filename)s:%(lineno)d] %(message)s"
}
......
......@@ -50,7 +50,7 @@ DEFAULT_LOGGING_CONFIG = {
def _configure_vllm_root_logger() -> None:
logging_config: Optional[Dict] = None
logging_config: Dict = {}
if not VLLM_CONFIGURE_LOGGING and VLLM_LOGGING_CONFIG_PATH:
raise RuntimeError(
......@@ -75,6 +75,11 @@ def _configure_vllm_root_logger() -> None:
type(custom_config).__name__)
logging_config = custom_config
for formatter in logging_config.get("formatters", {}).values():
# This provides backwards compatibility after #10134.
if formatter.get("class") == "vllm.logging.NewLineFormatter":
formatter["class"] = "vllm.logging_utils.NewLineFormatter"
if logging_config:
dictConfig(logging_config)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment