Skip to content

Support updating min_qps on vector search endpoints via PatchEndpoint#5584

Closed
janniklasrose wants to merge 1 commit intomainfrom
janniklasrose/vector-search-min-qps
Closed

Support updating min_qps on vector search endpoints via PatchEndpoint#5584
janniklasrose wants to merge 1 commit intomainfrom
janniklasrose/vector-search-min-qps

Conversation

@janniklasrose
Copy link
Copy Markdown

Adds min_qps as an optional+computed field on databricks_vector_search_endpoint. Updates call PatchEndpoint when min_qps changes; reads the value back from ScalingInfo.RequestedMinQps on each read. Also marks scaling_info as read-only to avoid ambiguity. Includes a unit test for the update path.

Co-authored-by: Isaac

Changes

Tests

  • make test run locally
  • relevant change in docs/ folder
  • covered with integration tests in internal/acceptance
  • using Go SDK
  • using TF Plugin Framework
  • has entry in NEXT_CHANGELOG.md file

Adds min_qps as an optional+computed field on databricks_vector_search_endpoint.
Updates call PatchEndpoint when min_qps changes; reads the value back from
ScalingInfo.RequestedMinQps on each read. Also marks scaling_info as read-only
to avoid ambiguity. Includes a unit test for the update path.

Co-authored-by: Isaac
@github-actions
Copy link
Copy Markdown
Contributor

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 5584
  • Commit SHA: a375f0700573d48331ca759f957c0503a5b7292d

Checks will be approved automatically on success.

@github-actions
Copy link
Copy Markdown
Contributor

Please ensure that the NEXT_CHANGELOG.md file is updated with any relevant changes.
If this is not necessary for your PR, please include the following in your PR description:
NO_CHANGELOG=true
and rerun the job.

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