Skip to content

Litellm tuesday cicd release final#20333

Merged
Sameerlite merged 6 commits intomainfrom
litellm_tuesday_cicd_release_final
Feb 3, 2026
Merged

Litellm tuesday cicd release final#20333
Sameerlite merged 6 commits intomainfrom
litellm_tuesday_cicd_release_final

Conversation

@Sameerlite
Copy link
Collaborator

Relevant issues

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

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

@vercel
Copy link

vercel bot commented Feb 3, 2026

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

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Feb 3, 2026 9:56am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 3, 2026

Greptile Overview

Greptile Summary

This PR reverts three previous features and fixes test failures in the MCP semantic tool filter tests.

Reverted Changes:

Test Fixes:

  • Fixed MCP semantic tool filter tests by calling filter_instance._build_router(tools) before filtering and initializing the metadata dict for the hook to store filter stats.
  • Added semantic-router import check with @pytest.mark.skipif to skip tests when the package is not installed.

Documentation:

  • Added four new MCP semantic filter configuration environment variables to the documentation.

Confidence Score: 4/5

  • This PR is safe to merge as it reverts previously committed features and fixes test failures
  • Score reflects that this is a revert PR for CICD stabilization. The reverts are clean and comprehensive, removing all related code. Test fixes are straightforward. Score is 4 instead of 5 because reverting features suggests the original implementations had issues, and the PR description lacks information about why these reverts were necessary or what problems they caused.
  • No files require special attention - all changes are clean reverts

Important Files Changed

Filename Overview
litellm/proxy/auth/model_checks.py Removed get_access_groups_from_models function and minor formatting changes
litellm/proxy/litellm_pre_call_utils.py Removed access group extraction logic for router filtering (reverted)
litellm/router.py Removed access group filtering from router deployment selection (reverted)
litellm/router_utils/common_utils.py Removed filter_deployments_by_access_groups function and minor formatting
litellm/router_utils/fallback_event_handlers.py Removed fallback list exhaustion check from fallback handler (reverted)
schema.prisma Removed VerificationToken table indexes from schema (reverted)

Sequence Diagram

sequenceDiagram
    participant Client
    participant ProxyServer as Proxy Server
    participant PreCallUtils as litellm_pre_call_utils
    participant Router
    participant Fallback as Fallback Handler
    participant LLM

    Note over ProxyServer,Router: REVERTED: Access Group Filtering
    Client->>ProxyServer: Request with API Key
    ProxyServer->>PreCallUtils: add_litellm_data_to_request()
    Note over PreCallUtils: ❌ REMOVED: Extract access groups<br/>from key/team models
    PreCallUtils->>Router: Request without access group metadata
    
    Note over Router: ❌ REMOVED: filter_deployments_by_access_groups()
    Router->>Router: Select deployment (no access group filtering)
    Router->>LLM: Forward request
    
    alt Request Fails
        LLM-->>Router: Error
        Router->>Fallback: run_async_fallback()
        Note over Fallback: ❌ REMOVED: Check if fallback list exhausted<br/>(fallback_depth >= fallback_group_length)
        Note over Fallback: ✅ KEPT: Only check max_fallbacks limit
        Fallback->>Router: Try next fallback model
        Router->>LLM: Retry with fallback
        LLM-->>Client: Response
    end
    
    Note over ProxyServer,Router: REVERTED: Database Indexes
    Note over ProxyServer: ❌ REMOVED: Prisma indexes on<br/>VerificationToken table<br/>(user_id, team_id, budget_reset_at, expires)
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@Sameerlite Sameerlite merged commit 793a7fd into main Feb 3, 2026
51 of 65 checks passed
@greptile-apps greptile-apps bot mentioned this pull request Feb 3, 2026
6 tasks
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