Skip to content

ops: suppress non-blocking LiteLLM P2028 alerts - #52

Closed
seonghobae wants to merge 1 commit into
mainfrom
codex/litellm-p2028-alert-suppression
Closed

ops: suppress non-blocking LiteLLM P2028 alerts#52
seonghobae wants to merge 1 commit into
mainfrom
codex/litellm-p2028-alert-suppression

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Summary

Adds an admin-scoped operational alert classifier so the known LiteLLM Prisma P2028 spend-log transaction timeout does not become a service outage/page event when it is explicitly non-blocking.

Closes #51.

What changed

Verification

  • python tests/test_operational_alerts.py
  • python tests/test_api_contract.py
  • python -m compileall contextual_orchestrator tests/test_operational_alerts.py
  • git diff --check
  • python -m pytest -q -> 153 passed

Notes

This does not mutate LiteLLM, PgCat, or PostgreSQL configuration. It prevents this specific non-blocking persistence-pressure signature from entering outage/page routing while leaving inference health, latency, 5xx, and customer-facing SLO alerts active.

@opencode-agent

Copy link
Copy Markdown
Contributor

OpenCode Review Overview

  • Head SHA: b6d73e5ca20ccb0183c9fa3561532df4939897fd
  • Workflow run: 29060639813
  • Workflow attempt: 1
  • Gate result: APPROVE (exit 0)

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file (5 files)"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file (5 files)"]
  R1 --> V1["required checks"]
  Evidence --> S2["Docs (2 files)"]
  S2 --> I2["operator or user guidance"]
  I2 --> R2["Review risk: Docs (2 files)"]
  R2 --> V2["docs review"]
  Evidence --> S3["Test: test_operational_alerts.py"]
  S3 --> I3["regression suite"]
  I3 --> R3["Review risk: Test: test_operational_alerts.py"]
  R3 --> V3["targeted test run"]
Loading

@seonghobae

Copy link
Copy Markdown
Contributor Author

Closing this branch because it modeled the problem as alert suppression. CodeGraph review shows the correct fix belongs in PR #46: usage/cost telemetry must be non-blocking and prompt-safe at the contextual-orchestrator boundary.

@seonghobae seonghobae closed this Jul 10, 2026
@seonghobae
seonghobae deleted the codex/litellm-p2028-alert-suppression branch July 10, 2026 01:27
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.

design: make usage telemetry non-blocking and prompt-safe

1 participant