Skip to content

Make the httpx logger less annoying when Transformers v5 is installed#30480

Merged
hmellor merged 1 commit intovllm-project:mainfrom
hmellor:silence-httpx
Dec 11, 2025
Merged

Make the httpx logger less annoying when Transformers v5 is installed#30480
hmellor merged 1 commit intovllm-project:mainfrom
hmellor:silence-httpx

Conversation

@hmellor
Copy link
Member

@hmellor hmellor commented Dec 11, 2025

Transfor ers v5 uses httpx to access the Hugging Face Hub. When its logger is set to INFO, the logging output is quite verbose.

This PR sets httpx's logging level to WARNING when vLLM's logging level is default (INFO) so that:

  • The default log level is less spammy
  • When users specify non-default logging levels, httpx will follow

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request aims to reduce the verbosity of the httpx logger by setting its level to WARNING when the main vLLM logger is at the default INFO level. The implementation correctly handles this case, but it fails to adjust the httpx logger level when a non-default level (like DEBUG) is set, which contradicts one of the stated goals of the PR. I've provided a suggestion to fix this logical issue to ensure httpx logging level matches the user's configuration for all non-default cases.

@hmellor hmellor enabled auto-merge (squash) December 11, 2025 11:49
@github-actions github-actions bot added the ready ONLY add when PR is ready to merge/full CI is needed label Dec 11, 2025
@hmellor hmellor merged commit 97a042f into vllm-project:main Dec 11, 2025
48 checks passed
@hmellor hmellor deleted the silence-httpx branch December 11, 2025 15:44
Majid-Taheri pushed a commit to Majid-Taheri/vllm that referenced this pull request Dec 23, 2025
…ed (vllm-project#30480)

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Ubuntu <mjtaheri68@gmail.com>
dsuhinin pushed a commit to dsuhinin/vllm that referenced this pull request Jan 21, 2026
…ed (vllm-project#30480)

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: dsuhinin <suhinin.dmitriy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants