Skip to content

Commit 2e003a2

Browse files
authored
Cleanup install documentation (#951)
## Changes This PR removes the gif showing old method of install UCX using install.sh. Also updating the Readme to show the new method to install Windows and added new image ### Functionality - [X] added relevant user documentation - [ ] added new CLI command - [ ] modified existing command: `databricks labs ucx ...` - [ ] added a new workflow - [ ] modified existing workflow: `...` - [ ] added a new table - [ ] modified existing table: `...` ### Tests - [ ] manually tested - [ ] added unit tests - [ ] added integration tests - [ ] verified on staging environment (screenshot attached)
1 parent b1173b7 commit 2e003a2

File tree

5 files changed

+2
-8
lines changed

5 files changed

+2
-8
lines changed

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,8 @@ For questions, troubleshooting or bug fixes, please see your Databricks account
7373
#### Install Databricks CLI on macOS
7474
![macos_install_databricks](docs/macos_1_databrickslabsmac_installdatabricks.gif)
7575

76-
#### Install Databricks CLI via curl on Windows
77-
![winos_install_databricks](docs/winos_1_databrickslabsmac_installdatabricks.gif)
76+
#### Install Databricks CLI on Windows
77+
![windows_install_databricks.png](docs/windows_install_databricks.png)
7878

7979
### Download & Install
8080

docs/ucx-install.gif

-636 KB
Binary file not shown.
302 KB
Loading
-58.1 KB
Binary file not shown.

tests/integration/azure/test_access.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,3 @@ def test_save_spn_permissions_local(ws, sql_backend, inventory_schema, make_rand
3838
az_res_perm = AzureResourcePermissions(installation, ws, AzureResources(ws, include_subscriptions=""), location)
3939
path = az_res_perm.save_spn_permissions()
4040
assert ws.workspace.get_status(path)
41-
42-
43-
def test_cli(ws):
44-
from databricks.labs.ucx.cli import save_storage_and_principal
45-
46-
save_storage_and_principal(ws)

0 commit comments

Comments
 (0)