Skip to content

Add direct-mode bundle support for AI Gateway model_provider_service - #6634

Open
alex-khakhlyuk wants to merge 3 commits into
aigov-dabs-mcp-servicefrom
aigov-dabs-model-provider-service
Open

Add direct-mode bundle support for AI Gateway model_provider_service#6634
alex-khakhlyuk wants to merge 3 commits into
aigov-dabs-mcp-servicefrom
aigov-dabs-model-provider-service

Conversation

@alex-khakhlyuk

Copy link
Copy Markdown
Member

Third and last of the stacked split of #6410 (stacked on the mcp_service PR). Direct engine only; provider_type is immutable (recreate) and write-only provider secrets + server-computed fields are suppressed to avoid phantom drift.

Changes

Why

Tests

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Approval status: pending

/acceptance/bundle/ - needs approval

30 files changed
Suggested: @janniklasrose
Also eligible: @denik, @andrewnester, @pietern, @shreyas-goenka, @anton-107, @lennartkats-db

/bundle/ - needs approval

21 files changed
Suggested: @janniklasrose
Also eligible: @denik, @andrewnester, @pietern, @shreyas-goenka, @anton-107, @lennartkats-db

General files (require maintainer)

42 files changed
Based on git history:

  • @janniklasrose -- recent work in bundle/direct/dresources/, bundle/internal/validation/generated/, libs/testserver/

Any maintainer (@andrewnester, @anton-107, @denik, @pietern, @shreyas-goenka, @simonfaltum, @renaudhartert-db, @janniklasrose, @lennartkats-db, @rugpanov, @rclarey) can approve all areas.
See OWNERS for ownership rules.

@alex-khakhlyuk
alex-khakhlyuk force-pushed the aigov-dabs-model-provider-service branch from 9a78c32 to eb8fe3b Compare September 11, 2026 13:33
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 1cd813f

Run: 34877925074

Env ❌​FAIL 🟨​KNOWN ✅​pass 🙈​skip Time
❌​ aws linux 2 1 276 26 7:23
❌​ aws windows 2 1 278 24 7:54
❌​ azure linux 2 1 275 26 7:23
❌​ azure windows 2 1 277 24 7:24
❌​ gcp linux 2 1 276 26 9:33
❌​ gcp windows 2 1 278 24 10:37
Test Name aws linux aws windows azure linux azure windows gcp linux gcp windows
🟨​ TestAccept 🟨​K 🟨​K 🟨​K 🟨​K 🟨​K 🟨​K
❌​ TestAccept/bundle/resources/model_provider_services/lifecycle ❌​F ❌​F ❌​F ❌​F ❌​F ❌​F
❌​ TestAccept/bundle/resources/model_provider_services/lifecycle/DATABRICKS_BUNDLE_ENGINE=direct/DMS=true ❌​F ❌​F ❌​F ❌​F ❌​F ❌​F

Comment thread bundle/direct/dresources/resources.yml Outdated
# rejects changing it via Update, so a local change must recreate.
- field: config.provider_type
reason: immutable
ignore_remote_changes:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

see #6654

@alex-khakhlyuk
alex-khakhlyuk force-pushed the aigov-dabs-model-provider-service branch from eb8fe3b to d1948b6 Compare September 14, 2026 12:51
@alex-khakhlyuk
alex-khakhlyuk force-pushed the aigov-dabs-model-provider-service branch 2 times, most recently from 5448793 to 1cd813f Compare September 14, 2026 17:55
alex-khakhlyuk and others added 3 commits September 14, 2026 19:21
Third and last of the stacked split of #6410 (stacked on the mcp_service PR). Direct engine only; provider_type is immutable (recreate) and write-only provider secrets + server-computed fields are suppressed to avoid phantom drift.

Co-authored-by: Isaac <no-reply@databricks.com>
Follow-up mirroring #6654 (model_services) and the mcp_services cleanup, now
that #6618 enabled generating field behaviors from the response type. Added
catalog.ModelProviderService to apitypes.yml, removed the hand-written
recreate_on_changes (config.provider_type) and ignore_remote_changes
(inference_table + write-only provider credentials) from resources.yml, and
regenerated resources.generated.yml. The generator now derives all of these
from the OpenAPI schema (and additionally picks up config.anthropic.relayed.
plan_type as immutable); no suppression is lost.

Co-authored-by: Isaac <no-reply@databricks.com>
…t_tag)

Main upgraded databricks-sdk-go v0.177 -> v0.178, the GA-sync SDK that dropped
`owner` (effective_owner only now), the request_tag rate-limit feature, and the
anthropic relayed plan_type from the AI Gateway securable types. Adapted the
model_provider_service resource to v0.178: removed the stale Owner field from
modelProviderServiceBody, and regenerated schema/validation/refschema/pydabs.
RATE_LIMIT_KEY_REQUEST_TAG is now absent from model_provider_services (and
model_services / mcp_services), matching the v0.178 spec.

Co-authored-by: Isaac <no-reply@databricks.com>
@alex-khakhlyuk
alex-khakhlyuk force-pushed the aigov-dabs-model-provider-service branch from 1cd813f to e95c91a Compare September 14, 2026 19:21
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.

3 participants