Skip to content

Resolve relative paths in variable overrides against the bundle root - #7049

Draft
denik wants to merge 5 commits into
mainfrom
denik/variable-overrides-root-paths
Draft

denik wants to merge 5 commits into
mainfrom
denik/variable-overrides-root-paths

Conversation

@denik

@denik denik commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Changes

Relative paths in .databricks/bundle/<target>/variable-overrides.json now resolve against the bundle root. Values from that file are located at <variable-overrides> in the bundle root, so diagnostics read in <variable-overrides>:4:6 instead of pointing at a file users don't edit. JSON parse errors still name the real file.

Before, paths resolved relative to .databricks/bundle/<target>/: task files failed with "path ... is defined relative to the bundle root directory", and libraries were silently rewritten under .databricks/bundle/<target>/.

Why

The overrides file is written by tools (not users); paths in it are relative to the bundle.

Tests

acceptance/bundle/validate/variable_overrides_file_complex_path (from #7029) now records both paths resolving against the bundle root and a warning located at <variable-overrides>; its Badness entry is removed.

Stacked on #7029.

This pull request and its description were written by Isaac.

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 789a917

Run: 38748875584

Env ✅​pass 🙈​skip Time
✅​ aws linux-2core-8gb 276 16 5:55
✅​ aws-windows-latest-4core-16gb 278 14 3:40
✅​ azure linux-2core-8gb 275 16 5:38
✅​ azure-windows-latest-4core-16gb 277 14 3:51
✅​ gcp linux-2core-8gb 276 16 5:31
✅​ gcp-windows-latest-4core-16gb 278 14 4:13
Top 6 slowest tests (at least 2 minutes):
duration env testname
4:09 gcp-windows-latest-4core-16gb TestAccept
4:05 aws linux-2core-8gb TestAccept
4:00 azure linux-2core-8gb TestAccept
3:47 azure-windows-latest-4core-16gb TestAccept
3:42 gcp linux-2core-8gb TestAccept
3:36 aws-windows-latest-4core-16gb TestAccept

Base automatically changed from denik/acc-config-edge-cases to main October 14, 2026 13:52
@denik
denik force-pushed the denik/variable-overrides-root-paths branch from 069c838 to 55314d6 Compare October 14, 2026 14:38
@github-actions github-actions Bot added the DABs DABs related issues label Oct 14, 2026
denik and others added 5 commits October 15, 2026 16:06
Values from .databricks/bundle/<target>/variable-overrides.json are located at
<variable-overrides> in the bundle root: the file is written by tools, so
relative paths in it resolve against the bundle root, and diagnostics name it
by that label. JSON parse errors still point at the real file.

Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
… removal (#7068)

Co-authored-by: Isaac <no-reply@databricks.com>
@denik
denik force-pushed the denik/variable-overrides-root-paths branch from 0271cf6 to 789a917 Compare October 15, 2026 14:08

This branch has not been deployed

No deployments
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.

2 participants