Skip to content

direct: Add model serving endpoints #3890

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 24 commits into
base: main
Choose a base branch
from
Draft

direct: Add model serving endpoints #3890

wants to merge 24 commits into from
+2,816 −24

Conversation

@shreyas-goenka
Copy link
Contributor

@shreyas-goenka shreyas-goenka commented Nov 6, 2025 •
edited
Loading

Changes

Adds support for model serving endpoints in direct mode with create, update, delete, and wait logic.

Tests

Added acceptance tests verifying field triggers:

  • Recreate tests: name, catalog_name, schema_name, table_name_prefix, route_optimized
  • Update tests: description, tags, served_entities

andrewnester
andrewnester approved these changes Nov 6, 2025
View reviewed changes
denik
denik reviewed Nov 6, 2025
View reviewed changes
@shreyas-goenka shreyas-goenka marked this pull request as draft November 6, 2025 16:52
@eng-dev-ecosystem-bot
Copy link
Collaborator

eng-dev-ecosystem-bot commented Nov 6, 2025 •
edited
Loading

Run: 19235905327

Env ❌​FAIL 🟨​KNOWN 💚​RECOVERED 🙈​SKIP ✅​pass 🙈​skip
💚​ aws linux 1 1 359 607
💚​ aws windows 1 1 360 606
❌​ azure linux 2 1 1 357 606
❌​ azure windows 2 1 1 358 605
Test Name aws linux aws windows azure linux azure windows
TestAccept 💚​R 💚​R 🟨​K 🟨​K
TestAccept/bundle/resources/model_serving_endpoints/basic ✅​p ✅​p ❌​F ❌​F
TestAccept/bundle/resources/model_serving_endpoints/basic/DATABRICKS_BUNDLE_ENGINE=terraform ✅​p ✅​p ❌​F ❌​F
TestAccept/bundle/run/app-with-job 🙈​S 🙈​S 🙈​S 🙈​S

@shreyas-goenka shreyas-goenka deployed to test-trigger-is November 10, 2025 15:31 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@denik denik denik left review comments

@andrewnester andrewnester andrewnester approved these changes

@anton-107 anton-107 Awaiting requested review from anton-107 anton-107 is a code owner

@pietern pietern Awaiting requested review from pietern pietern is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@shreyas-goenka @eng-dev-ecosystem-bot @denik @andrewnester