Skip to content

Add Resources.ResolveKey for resolving key or type.key - #7128

Merged
denik merged 2 commits into
mainfrom
denik/resolve-key
Oct 21, 2026
Merged

denik merged 2 commits into
mainfrom
denik/resolve-key

Conversation

@denik

@denik denik commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Changes

One resolver for a resource given as key or type.key, replacing three copies: ResolveSelect, and FindResourceByConfigKey used by bundle deployment bind/unbind and apps import. bind and unbind now also accept type.key.

Why

Preparation for lifecycle.depends_on, which needs the same resolution: #7120 (and #7132 on top of it).

This pull request and its description were written by Isaac.

denik and others added 2 commits October 20, 2026 15:31
Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
@github-actions github-actions Bot added the DABs DABs related issues label Oct 20, 2026
@denik
denik marked this pull request as ready for review October 20, 2026 13:43
@denik
denik requested review from a team as code owners October 20, 2026 13:43
@denik
denik requested a review from andrewnester October 20, 2026 13:43
@denik
denik added this pull request to stack #7131 October 20, 2026 13:48
@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: bffe239

Run: 39431340100

Env ✅​pass 🙈​skip Time
✅​ aws linux-2core-8gb 280 15 4:34
✅​ aws-windows-latest-4core-16gb 282 13 3:51
✅​ azure linux-2core-8gb 279 15 4:40
✅​ azure-windows-latest-4core-16gb 281 13 3:32
✅​ gcp linux-2core-8gb 280 15 4:16
✅​ gcp-windows-latest-4core-16gb 282 13 3:39
Top 3 slowest tests (at least 2 minutes):
duration env testname
3:44 aws-windows-latest-4core-16gb TestAccept
3:33 gcp-windows-latest-4core-16gb TestAccept
3:26 azure-windows-latest-4core-16gb TestAccept

@denik
denik added this pull request to the merge queue Oct 21, 2026
Merged via the queue into main with commit 92df5a7 Oct 21, 2026
27 checks passed
@denik
denik deleted the denik/resolve-key branch October 21, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DABs DABs related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants