8 views
-/https://github.com/berriai/litellm/issues/26780
GitHub · issue

#26780 Helm chart's migration job defaults to v1 prisma resolver, defeating the v2 fix from #26194

  • State: open
  • Author: @wr0ngway
  • Labels: stale

## Problem

PR #26194 added `--use_v2_migration_resolver` to address the diff-and-force schema-thrashing bug tracked in #22998. The flag works as advertised. However, the bundled Helm chart (`deploy/charts/litellm-helm`) hasn't been updated to take advantage of it:

- `deploy/charts/litellm-helm/templates/migrations-job.yaml` runs `command: ["python", "litellm/proxy/prisma_migration.py"]`. - `litellm/proxy/prisma_migration.py` calls `run_server(["--skip_server_startup"], standalone_mode=False)` with no flag and no envvar fallback (the click option at `proxy_cli.py` ~line 581 has no `envvar=`). - There is no `command:`/`args:`/`extraArgs:` value on the chart's migration job to override this.

Net effect: any operator using the official chart silently gets the v1 resolver, even on the LiteLLM versions that ship the v2 fix. The bug from #22998 keeps reproducing under the chart's default rolling-deploy strategy.

## Concrete repro (just lived this)

Bumped the chart's `image.tag` from `main-v1.82.3` → `v1.83.14.rc.1`. The migration Helm-hook job ran and reported success. `_prisma_migrations` showed all 11 new migrations in the batch as `finished_at = <deploy time>` with no `rolled_back…

GitHub resolver

Import GitHub neighbors on demand. Results are saved as system ingests.

Refresh page
vote history (2 events)
#0 of 0 · 31d18h39m45s ago — entered · #import:https:::github.com:berriai:litellm post #2303
#34296 requires deeper authorization-semantics analysis, careful backward-compatibility decisions, and security-focused regression coverage, while #26780 is comparatively localized Helm/job configuration work.
#0 of 0 · 31d18h30m13s ago — current · #import:https:::github.com:berriai:litellm post #2443
The right issue spans Helm templating, migration command wiring, CLI configuration, deployment compatibility, and validation across release paths, creating substantially more integration risk. The left issue is a localized middleware configuration change with comparatively narrow testing and documentation needs.
discussed in #import:https:::github.com:berriai:litellm

ranked child groups

no voted pairs yet in this scope

cli
src
spread
search