direct: model_serving: ignore budget_policy_id, rate_limits at plan level (#4638)

denik · web-flow · commit b634235c168a · 2026-03-04T11:22:09.000+01:00
## Why Previously changing these fields would result in planned 'update' but DoUpdate is actually no-op for these, see https://github.com/databricks/cli/blob/10d976b5d086fb8993532bb5640b32cbd08fa106/bundle/direct/dresources/model_serving_endpoint.go#L288 Now, this will be skipped at plan level with appropriate reason. ## Tests No new tests, this a config-only change.
diff --git a/NEXT_CHANGELOG.md b/NEXT_CHANGELOG.md
@@ -6,6 +6,7 @@
 
 ### Bundles
 * Validate that either source_code_path or git_source is set for apps ([#4632](https://github.com/databricks/cli/pull/4632))
+* direct: model_serving_endpoints: ignore budget_policy_id, rate_limits changes in the plan. Previously they were ignored as well but action as "update". ([#4638](https://github.com/databricks/cli/pull/4638))
 
 ### Dependency updates
 
diff --git a/bundle/direct/dresources/README.md b/bundle/direct/dresources/README.md
@@ -10,6 +10,7 @@
  - The logic what kind of update it is should be in FieldTriggers / ClassifyChange methods. The methods performing update should not have logic in them on what method to call.
  - Create/Update/Delete methods should not need to read any state. (We can implement support for passing remoteState we already to these methods if such need arises though).
  - Prefer “with refresh” variants of methods if resource API supports that. That avoids explicit DoRead() call.
+ - For update with complex logic, ensure that DoUpdate() does not result in no-op. If certain fields could not be updated, they should be excluded at plan level in resources.yml.
 
 Nice to have
  - Add link to corresponding API documentation before each method.
diff --git a/bundle/direct/dresources/resources.yml b/bundle/direct/dresources/resources.yml
@@ -227,6 +227,16 @@ resources:
       # Routes have custom SuppressDiff (lines 387-388)
       - field: config.traffic_config
         reason: managed
+      - field: budget_policy_id
+        reason: no_update_api
+        # There is PublicPreview endpoints but we're not using it https://docs.databricks.com/api/workspace/servingendpoints/put
+      - field: rate_limits
+        reason: not_implemented
+    ignore_local_changes:
+      - field: budget_policy_id
+        reason: no_update_api
+      - field: rate_limits
+        reason: not_implemented
     backend_defaults:
       # https://github.com/databricks/terraform-provider-databricks/blob/4eba541abe1a9f50993ea7b9dd83874207e224a1/serving/resource_model_serving.go#L383
       # common.CustomizeSchemaPath(m, "config", "served_entities", "name").SetComputed()
