Skip to content

[Fix] Sorting for /v2/model/info#19971

Merged
yuneng-jiang merged 1 commit intomainfrom
litellm_v2_model_info_sorting_fix
Jan 29, 2026
Merged

[Fix] Sorting for /v2/model/info#19971
yuneng-jiang merged 1 commit intomainfrom
litellm_v2_model_info_sorting_fix

Conversation

@yuneng-jiang
Copy link
Collaborator

Relevant issues

This PR fixes an issue with the /v2/model/info endpoint, where when passing in sortBy = created_at or updated_at, it will throw an error about comparing offset-naive and offset-aware timezones. This PR fixes this with normalizing time zones before comparing them.

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🐛 Bug Fix
✅ Test

Changes

image

@vercel
Copy link

vercel bot commented Jan 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
litellm Error Error Jan 29, 2026 2:09am

Request Review

@yuneng-jiang yuneng-jiang merged commit bc23a97 into main Jan 29, 2026
46 of 63 checks passed
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.

1 participant