Skip to content

Fix bug that v2 API logs are not displayed as expected - #789

Merged
vincent-olivert-riera merged 1 commit into
line:masterfrom
hoangpn:bug_fix/v2_API_logs_not_shown
Sep 28, 2026
Merged

vincent-olivert-riera merged 1 commit into
line:masterfrom
hoangpn:bug_fix/v2_API_logs_not_shown

Conversation

@hoangpn

@hoangpn hoangpn commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

In Promgen, the v2 API logs should be displayed by default because V2_API_LOGGING_ENABLED is set to True by default. However, these logs were set at the INFO level, while Python's default logging level is WARNING. To address this issue, we have configured the logger for PromgenMiddleware to use the INFO level specifically and added a StreamHandler for it.

@vincent-olivert-riera vincent-olivert-riera left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@hoangpn , please rebase it on master.

In Promgen, the v2 API logs should be displayed by default because V2_API_LOGGING_ENABLED
is set to True by default. However, these logs were set at the INFO level, while Python's
default logging level is WARNING. To address this issue, we have configured the logger for
PromgenMiddleware to use the INFO level specifically and added a StreamHandler for it.
@hoangpn
hoangpn force-pushed the bug_fix/v2_API_logs_not_shown branch from 8692952 to 74ad389 Compare September 28, 2026 01:06
@vincent-olivert-riera
vincent-olivert-riera merged commit d990023 into line:master Sep 28, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants