Skip to content

Conversation

fjakobs
Copy link
Contributor

@fjakobs fjakobs commented Aug 29, 2025

When running DABS from within a Databricks workspace with files under /Workspace/, the CLI would incorrectly use WSFS extensions even when deploying to a different workspace. This caused "path does not exist" errors because it tried to read files from the current workspace while deploying to the target workspace.

The fix adds cross-workspace deployment detection by comparing the current workspace host (DATABRICKS_HOST env var) with the target workspace host from the bundle configuration. When hosts differ, WSFS extensions are skipped and the standard workspace files client is used instead.

This resolves customer issues where dynamically generated DABS configurations work within the same workspace but fail when targeting a different workspace.

🤖 Generated with Claude Code

Changes

Why

Tests

When running DABS from within a Databricks workspace with files under
/Workspace/, the CLI would incorrectly use WSFS extensions even when
deploying to a different workspace. This caused "path does not exist"
errors because it tried to read files from the current workspace while
deploying to the target workspace.

The fix adds cross-workspace deployment detection by comparing the
current workspace host (DATABRICKS_HOST env var) with the target
workspace host from the bundle configuration. When hosts differ,
WSFS extensions are skipped and the standard workspace files client
is used instead.

This resolves customer issues where dynamically generated DABS
configurations work within the same workspace but fail when targeting
a different workspace.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@fjakobs fjakobs temporarily deployed to test-trigger-is August 29, 2025 10:51 — with GitHub Actions Inactive
@fjakobs fjakobs marked this pull request as draft August 29, 2025 10:53
@eng-dev-ecosystem-bot
Copy link
Collaborator

eng-dev-ecosystem-bot commented Aug 29, 2025

Run: 17321862651

Env ✅​pass 🔄​flaky 🙈​skip
✅​ aws windows 309 502
✅​ aws-ucws linux 421 400
✅​ aws-ucws windows 422 399
🔄​ azure linux 305 3 502
🔄​ azure windows 305 4 501
✅​ azure-ucws linux 421 399
✅​ azure-ucws windows 422 398
🔄​ gcp linux 303 4 504
✅​ gcp windows 308 503
10 failing tests:
Test Name azure linux azure windows gcp linux
TestAccept 🔄​flaky ✅​pass ✅​pass
TestAccept/bundle/deploy/pipeline/allow-duplicate-names/DATABRICKS_CLI_DEPLOYMENT=direct-exp ✅​pass 🔄​flaky ✅​pass
TestAccept/bundle/deploy/pipeline/allow-duplicate-names/DATABRICKS_CLI_DEPLOYMENT=terraform ✅​pass 🔄​flaky ✅​pass
TestAccept/bundle/deploy/pipeline/auto-approve ✅​pass 🔄​flaky ✅​pass
TestAccept/bundle/deploy/pipeline/auto-approve/DATABRICKS_CLI_DEPLOYMENT=direct-exp ✅​pass 🔄​flaky ✅​pass
TestAccept/bundle/resources/pipelines/update/DATABRICKS_CLI_DEPLOYMENT=direct-exp ✅​pass ✅​pass 🔄​flaky
TestAccept/bundle/resources/pipelines/update/DATABRICKS_CLI_DEPLOYMENT=terraform ✅​pass ✅​pass 🔄​flaky
TestAccept/bundle/templates/default-python/integration_classic 🔄​flaky ✅​pass 🔄​flaky
TestAccept/bundle/templates/default-python/integration_classic/DATABRICKS_CLI_DEPLOYMENT=terraform/UV_PYTHON=3.10 🔄​flaky ✅​pass ✅​pass
TestAccept/bundle/templates/default-python/integration_classic/DATABRICKS_CLI_DEPLOYMENT=terraform/UV_PYTHON=3.13 ✅​pass ✅​pass 🔄​flaky

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