Fix log levels: info for unconfigured, warning for misconfigured

Suppress noisy error log fired every cron tick when spend log cleanup
is simply not configured. _should_delete_spend_logs already logs the
specific reason at the right level (info for None, warning for
invalid value), so the redundant blanket error log in
cleanup_old_spend_logs is removed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
yuneng-jiang
2026-03-03 10:21:31 -08:00
co-authored by Claude Sonnet 4.6
parent 224c617119
commit a1ba6c9fa6
@@ -60,7 +60,7 @@ class SpendLogCleanup:
)
return True
except ValueError as e:
verbose_proxy_logger.error(
verbose_proxy_logger.warning(
f"Invalid maximum_spend_logs_retention_period value: {retention_setting}, error: {str(e)}"
)
return False
@@ -121,9 +121,6 @@ class SpendLogCleanup:
verbose_proxy_logger.info(f"Cleanup job triggered at {datetime.now()}")
if not self._should_delete_spend_logs():
verbose_proxy_logger.error(
"Skipping cleanup — invalid or missing retention setting."
)
return
if self.retention_seconds is None: