Skip to content

fix(gateway): use consistent logger in budget.py#929

Merged
njbrake merged 1 commit intomainfrom
fix/budget-consistent-logger
Mar 13, 2026
Merged

fix(gateway): use consistent logger in budget.py#929
njbrake merged 1 commit intomainfrom
fix/budget-consistent-logger

Conversation

@njbrake
Copy link
Copy Markdown
Contributor

@njbrake njbrake commented Mar 13, 2026

Description

Replaces logging.getLogger(__name__) with the shared logger from any_llm.gateway.log_config, matching all other gateway modules. Ensures budget log messages go through the same Rich formatter and configuration.

PR Type

  • 🐛 Bug Fix

Relevant issues

Fixes #918

Checklist

  • I understand the code I am submitting.
  • I have added unit tests that prove my fix/feature works
  • I have run this code locally and verified it fixes the issue.
  • New and existing tests pass locally
  • Documentation was updated where necessary
  • I have read and followed the contribution guidelines
  • AI Usage:
    • No AI was used.
    • AI was used for drafting/refactoring.
    • This is fully AI-generated.

AI Usage Information

  • AI Model used: Claude Opus 4.6

  • AI Developer Tool used: Claude Code

  • I am an AI Agent filling out this form (check box if true)

Replace logging.getLogger(__name__) with the shared logger from
log_config, matching all other gateway modules. Ensures budget
log messages go through the same Rich formatter and configuration.

Fixes #918

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@njbrake njbrake temporarily deployed to integration-tests March 13, 2026 14:23 — with GitHub Actions Inactive
@njbrake njbrake temporarily deployed to integration-tests March 13, 2026 14:23 — with GitHub Actions Inactive
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 42 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@njbrake njbrake merged commit 80b3c6b into main Mar 13, 2026
14 checks passed
@njbrake njbrake deleted the fix/budget-consistent-logger branch March 13, 2026 14:35
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.

fix(gateway): use consistent logger in budget.py

1 participant