Skip to content

refactor: drop chat_model_configs provider column (#26877)#26962

Open
github-actions[bot] wants to merge 1 commit into
release/2.35from
backport/26877-to-2.35
Open

refactor: drop chat_model_configs provider column (#26877)#26962
github-actions[bot] wants to merge 1 commit into
release/2.35from
backport/26877-to-2.35

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

Cherry-pick of #26877

Original PR: #26877 — refactor: drop chat_model_configs provider column
Merge commit: 047c474
Requested by: @mafredri

The provider type already lives authoritatively in ai_providers.type,
reachable on every active row through ai_provider_id, which the
chat_model_configs_ai_provider_required_when_active CHECK makes
mandatory. The stored provider string was a denormalized copy the system
kept in sync with a startup backfill and no longer needs.

Every surface now derives provider type from the linked ai_providers
row. Telemetry is the one exception: it keeps emitting provider, now
sourced from ai_providers.type via a JOIN, so the BigQuery column and the
Nexus dashboards that read it are unaffected. The experimental HTTP/SDK
response drops provider and makes ai_provider_id required, since those
endpoints return only active configs; consumers resolve provider type
from ai_provider_id and the AI providers listing.

This ships in a single release with no compatibility window: production
reads the table via SELECT *, so a pre-drop binary fails config reads the
moment the column is gone. Operators must scale to zero before upgrading,
and there is no rollback.

Closes CODAGT-599

(cherry picked from commit 047c474)
@github-actions github-actions Bot added the cherry-pick/v2.35 Cherry-pick PR targeting release/2.35 label Jul 2, 2026
@github-actions github-actions Bot requested a review from mafredri July 2, 2026 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick/v2.35 Cherry-pick PR targeting release/2.35

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant