Skip to content

chore: Include litellm v readiness#180

Merged
noahpodgurski merged 4 commits into
mainfrom
include-litellm-v-readiness
Jun 17, 2026
Merged

chore: Include litellm v readiness#180
noahpodgurski merged 4 commits into
mainfrom
include-litellm-v-readiness

Conversation

@noahpodgurski

@noahpodgurski noahpodgurski commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

What's new

  • LiteLLM updated their readiness to not include the version anymore
  • Re-include it in MLPA /health/readiness by calling public/model_hub/info
  • Fix warnings, make tests more consistent

Before:

{
  "status": "connected",
  "mlpa_version": "1.0.12",
  "pg_server_dbs": {
    "postgres": "connected",
    "app_attest": "connected"
  },
  "litellm": {
    "status": "healthy",
    "db": "connected"
  }
}

After:

{
  "status": "connected",
  "mlpa_version": "1.0.12",
  "pg_server_dbs": {
    "postgres": "connected",
    "app_attest": "connected"
  },
  "litellm": {
    "litellm_version": "1.84.4",
    "status": "healthy",
    "db": "connected"
  }
}

https://mozilla-hub.atlassian.net/browse/AIPLAT-1035

@noahpodgurski noahpodgurski requested a review from a team as a code owner June 16, 2026 13:11
Comment thread src/mlpa/core/routers/health/health.py Outdated
Comment thread src/mlpa/core/routers/health/health.py Outdated
@noahpodgurski noahpodgurski merged commit 65ad2d2 into main Jun 17, 2026
1 check failed
@noahpodgurski noahpodgurski deleted the include-litellm-v-readiness branch June 17, 2026 13:07
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