Uh oh!
There was an error while loading. Please reload this page.
Add more schema defaults - #5865
Merged
Merged
Conversation
janniklasrosetemporarily deployed
to
test-trigger-is
July 8, 2026 14:26 — with 
GitHub Actions
Inactive
janniklasrosetemporarily deployed
to
test-trigger-is
July 8, 2026 14:26 — with GitHub Actions
Inactive
radakam
reviewed
Jul 8, 2026
Uh oh!
There was an error while loading. Please reload this page.
Collaborator
Integration test reportCommit: 93e57c6
8 interesting tests: 4 SKIP, 2 flaky, 1 RECOVERED, 1 KNOWN
Top 7 slowest tests (at least 2 minutes):
|
The previous commit had a stray double-quote in the resources.yml backend_defaults field paths (so they never matched), re-listed keys that were already present, and duplicated the iceberg catalogManaged key in the testserver map literal (a compile error). Align the backend_defaults list with the testserver defaults and refresh the golden output. Co-authored-by: Isaac
janniklasrosetemporarily deployed
to
test-trigger-is
July 8, 2026 16:20 — with GitHub Actions
Inactive
janniklasrosetemporarily deployed
to
test-trigger-is
July 8, 2026 16:20 — with GitHub Actions
Inactive
janniklasrosetemporarily deployed
to
test-trigger-is
July 8, 2026 16:45 — with GitHub Actions
Inactive
janniklasrosetemporarily deployed
to
test-trigger-is
July 8, 2026 16:45 — with GitHub Actions
Inactive
janniklasrose
enabled auto-merge
July 8, 2026 17:53
pietern
approved these changes
Jul 8, 2026
eng-dev-ecosystem-bot
commented
Jul 8, 2026
Collaborator
Integration test reportCommit: 5411bc8
36 interesting tests: 21 flaky, 9 FAIL, 3 RECOVERED, 2 SKIP, 1 KNOWN
Top 50 slowest tests (at least 2 minutes):
|
deco-sdk-taggingBot
added a commit
that referenced
this pull request
Jul 9, 2026
## Release v1.7.0 ### CLI * An explicitly selected profile (`--profile` or a bundle's `workspace.profile`) now takes precedence over auth environment variables (`DATABRICKS_HOST`, `DATABRICKS_TOKEN`, etc.) instead of being silently shadowed by them; env vars still fill auth fields the profile leaves empty ([#5096](#5096)). * Fix intermittent crashes when processing pages from API calls ([#5815](#5815)). ### Bundles * direct: add basic version of job_runs resource (experimental) ([#5603](#5603)). * Fix permissions added to a job or pipeline by a Python (PyDABs) mutator failing to deploy with "must have exactly one owner"; the deploying identity is now set as owner, matching resources whose permissions are declared in YAML ([#5821](#5821)). * Remove duplicate enum values for jsonschema.json ([#5839](#5839)). * direct: volumes: support `volume_path` property ([#5550](#5550)). * direct: Fix deploy bug when a `postgres_projects`, `postgres_branches`, or `postgres_endpoints` field is set to its zero value (e.g. `enable_pg_native_login: false`, `replace_existing: false`) ([#5782](#5782)). * `bundle run --only` help now documents the `+` modifier syntax: prefix a task key with `+` to also run its upstream tasks, or suffix it with `+` for downstream tasks ([#5760](#5760)). * direct: Recognize UC-managed catalog and schema property defaults to avoid unnecessary drift ([#5865](#5865) & [#5870](#5870)). * Fix `bundle deploy --select <resource>` skipping the resource's grants and permissions; they are now applied as part of the selected resource ([#5852](#5852)). * Support `purge_on_delete: true` on `postgres_branches` so bundles can hard-delete a Lakebase branch on destroy (skipping the soft-delete retention window) ([#5801](#5801)). * Support `replace_existing: true` on `postgres_databases` and `postgres_roles` so bundles can take over a database or role that already exists on a Lakebase branch instead of failing with `ALREADY_EXISTS` ([#5803](#5803)). ### Dependency updates * Bump databricks-sdk-go to v0.154.0 ([#5855](#5855)). * Bump terraform-provider to 1.121.0 ([#5857](#5857)). * Bump OpenTelemetry dependencies to v1.44.0 to address [CVE-2026-41178](GHSA-5wrp-cwcj-q835) ([#5873](#5873)).
RCoff pushed a commit
to RCoff/databricks-cli
that referenced
this pull request
Jul 10, 2026
…lta.feature.catalogManaged (databricks#5870) Follow-up to databricks#5865, which added the schema keys but did not cover catalogs or the `delta.feature.catalogManaged` key. This pull request and its description were written by Isaac, an AI coding agent. Co-authored-by: Isaac <isaac@example.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Add more UC-managed schema properties to the direct engine's
backend_defaultslist (and to the testserver mock that mirrors them), so the diff engine skips them when the backend populates them after create. Also fixes a stray double-quote typo, duplicatebackend_defaultsentries, and a duplicate map key that broke the build in the previous commit.Why
UC auto-populates a set of
unity.catalog.managed.*schema properties after create. Without listing them as backend defaults, every subsequent plan produces an empty Update that UC rejects with "UpdateSchema Nothing to update". The initial commit only covered two of these keys and introduced typos/duplicates.Tests
Extended acceptance coverage in
bundle/resources/schemas/drift/managed_properties; golden output regenerated../task fmt,./task lint(changed package), and the schema-drift acceptance tests pass.This PR was written by Claude Code.