Skip to content

fix: downgrade AuthConfig partial-init log from WARNING to INFO #1052

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
anatolykoptev wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: downgrade AuthConfig partial-init log from WARNING to INFO #1052

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions src/memos/configs/mem_scheduler.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -250,8 +250,12 @@ def validate_partial_initialization(self) -> "AuthConfig":
"All configuration components are None. This may indicate missing environment variables or configuration files."
)
elif failed_components:
logger.warning(
f"Failed to initialize components: {', '.join(failed_components)}. Successfully initialized: {', '.join(initialized_components)}"
# Use info level: individual from_local_env() methods already log
# warnings for actual initialization failures. Components that are
# simply not configured (no env vars) are not errors.
logger.info(
f"Components not configured: {', '.join(failed_components)}. "
f"Successfully initialized: {', '.join(initialized_components)}"
)

return self
Expand Down