-
Notifications
You must be signed in to change notification settings - Fork 87
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
Upgrade Databricks SDK to 0.30 #2429
Conversation
Dabricks Profile With training slash: [hsdb] With token and port appended [hsdb2] Without slash [hsdbuc] Test with SDK 0.29 Test with SDK 0.30 |
❌ 105/108 passed, 3 failed, 20 skipped, 1h24m13s total ❌ test_running_real_workflow_linter_job: databricks.labs.blueprint.parallel.ManyError: Detected 2 failures: ValueError: Invalid semantic version: 0.33.1+520240815173351 (3m4.032s)
❌ test_running_real_validate_groups_permissions_job: databricks.labs.blueprint.parallel.ManyError: Detected 2 failures: ValueError: Invalid semantic version: 0.33.1+520240815173902 (2m52.407s)
❌ test_running_real_migrate_groups_job: databricks.labs.blueprint.parallel.ManyError: Detected 2 failures: ValueError: Invalid semantic version: 0.33.1+520240815174015 (2m23.834s)
Running from acceptance #5232 |
can you finish this PR instead - #2417 |
Changes
Databricks SDK version 0.30 has fixed the bug due to incorrectly identifying cloud value when using :443 (databricks/databricks-sdk-py#656) or trailing slash (databricks/databricks-sdk-py#661) in the profile.
Upgrading SDK also had cascading effect on other API calls in UCX mainly on redash API. This PR also updates the relevant code.
Resolves #1757
Functionality
databricks labs ucx ...
...
...
Tests