Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve error handling for /Volumes paths in mode: development #1716

Merged
merged 2 commits into from
Aug 28, 2024

Conversation

lennartkats-db
Copy link
Contributor

Changes

  • Provide a more helpful error when using an artifact_path based on /Volumes
  • Allow the use of short_names in /Volumes paths

Example cases

Example of a valid /Volumes artifact_path:

  • artifact_path: /Volumes/catalog/schema/${workspace.current_user.short_name}/libs

Example of an invalid /Volumes path (when using mode: development):

  • artifact_path: /Volumes/catalog/schema/libs
  • Resulting error: artifact_path should contain the current username or ${workspace.current_user.short_name} to ensure uniqueness when using 'mode: development'

@andrewnester andrewnester added this pull request to the merge queue Aug 28, 2024
Merged via the queue into databricks:main with commit 85459c1 Aug 28, 2024
5 checks passed
andrewnester added a commit that referenced this pull request Aug 28, 2024
CLI:
 * Disable prompt for storage-credentials get command ([#1723](#1723)).

Bundles:
 * Do not treat empty path as a local path ([#1717](#1717)).
 * Correctly mark PyPI package name specs with multiple specifiers as remote libraries ([#1725](#1725)).
 * Improve error handling for /Volumes paths in mode: development ([#1716](#1716)).

Internal:
 * Revert hc-install version to 0.7.0 ([#1711](#1711)).
 * Ignore CLI version check on development builds of the CLI ([#1714](#1714)).

API Changes:
 * Added `databricks resource-quotas` command group.
 * Added `databricks policy-compliance-for-clusters` command group.
 * Added `databricks policy-compliance-for-jobs` command group.

OpenAPI commit 3eae49b444cac5a0118a3503e5b7ecef7f96527a (2024-08-21)
Dependency updates:
 * Bump github.com/databricks/databricks-sdk-go from 0.44.0 to 0.45.0 ([#1719](#1719)).
github-merge-queue bot pushed a commit that referenced this pull request Aug 28, 2024
CLI:
* Disable prompt for storage-credentials get command
([#1723](#1723)).

Bundles:
* Do not treat empty path as a local path
([#1717](#1717)).
* Correctly mark PyPI package name specs with multiple specifiers as
remote libraries ([#1725](#1725)).
* Improve error handling for /Volumes paths in mode: development
([#1716](#1716)).

Internal:
* Ignore CLI version check on development builds of the CLI
([#1714](#1714)).

API Changes:
 * Added `databricks resource-quotas` command group.
 * Added `databricks policy-compliance-for-clusters` command group.
 * Added `databricks policy-compliance-for-jobs` command group.

OpenAPI commit 3eae49b444cac5a0118a3503e5b7ecef7f96527a (2024-08-21)
Dependency updates:
* Bump github.com/databricks/databricks-sdk-go from 0.44.0 to 0.45.0
([#1719](#1719)).
* Revert hc-install version to 0.7.0
([#1711](#1711)).
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.

3 participants