Skip to content

direct: wildcard-match UC managed property backend defaults#5877

Open
radakam wants to merge 1 commit into
mainfrom
wildcard-uc-managed-property-defaults
Open

direct: wildcard-match UC managed property backend defaults#5877
radakam wants to merge 1 commit into
mainfrom
wildcard-uc-managed-property-defaults

Conversation

@radakam

@radakam radakam commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Use a single wildcard backend-default rule for UC-managed catalog/schema properties: properties['unity.catalog.managed.*.defaults.*']. This replaces per-key entries and covers current and future Auto Upgrade defaults without listing each property.

Why

UC fills these keys after create. If we don't treat them as backend defaults, plans keep trying empty updates and UC rejects them.

Replace per-key backend_defaults entries for catalogs and schemas with a
single unity.catalog.managed.*.defaults.* pattern so new Auto Upgrade keys
are recognized without listing each one explicitly.
@radakam radakam force-pushed the wildcard-uc-managed-property-defaults branch from 4db5696 to ad7c34f Compare July 9, 2026 10:58
@radakam radakam temporarily deployed to test-trigger-is July 9, 2026 11:00 — with GitHub Actions Inactive
@radakam radakam temporarily deployed to test-trigger-is July 9, 2026 11:00 — with GitHub Actions Inactive
@radakam radakam marked this pull request as ready for review July 9, 2026 11:02
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Approval status: pending

/bundle/ - needs approval

Files: bundle/direct/bundle_plan_test.go, bundle/direct/dresources/resources.yml
Suggested: @denik
Also eligible: @janniklasrose, @pietern, @andrewnester, @anton-107, @shreyas-goenka, @lennartkats-db

General files (require maintainer)

Files: NEXT_CHANGELOG.md, libs/structs/structpath/path.go, libs/structs/structpath/path_test.go
Based on git history:

  • @denik -- recent work in libs/structs/structpath/, bundle/direct/dresources/, ./

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

@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: ad7c34f

Run: 29013447069

Env 💚​RECOVERED 🙈​SKIP ✅​pass 🙈​skip Time
💚​ aws linux 4 4 230 1077 5:01
💚​ aws windows 4 4 232 1075 7:20
💚​ aws-ucws linux 4 4 314 995 8:44
💚​ aws-ucws windows 4 4 316 993 8:13
💚​ azure linux 4 4 230 1076 5:13
💚​ azure windows 4 4 232 1074 7:07
💚​ azure-ucws linux 4 4 316 992 8:44
💚​ azure-ucws windows 4 4 318 990 7:41
💚​ gcp linux 4 4 229 1078 5:09
💚​ gcp windows 4 4 231 1076 6:48
8 interesting tests: 4 RECOVERED, 4 SKIP
Test Name aws linux aws windows aws-ucws linux aws-ucws windows azure linux azure windows azure-ucws linux azure-ucws windows gcp linux gcp windows
💚​ TestAccept 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
🙈​ TestAccept/bundle/invariant/no_drift 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_endpoints/drift/recreated_same_name 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_indexes/recreate/embedding_dimension 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/ssh/connection 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
💚​ TestFetchRepositoryInfoAPI_FromRepo 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
💚​ TestFetchRepositoryInfoAPI_FromRepo/root 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
💚​ TestFetchRepositoryInfoAPI_FromRepo/subdir 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
Top 10 slowest tests (at least 2 minutes):
duration env testname
6:06 azure-ucws windows TestAccept
6:05 aws windows TestAccept
5:59 gcp windows TestAccept
5:59 azure windows TestAccept
5:57 aws-ucws windows TestAccept
2:53 gcp linux TestAccept
2:52 aws linux TestAccept
2:52 azure linux TestAccept
2:50 azure-ucws linux TestAccept
2:50 aws-ucws linux TestAccept

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.

2 participants