Skip to content

Commit 24d4acb

Browse files
authored
Fix installation issue when upgrading from an older version of the tool and improve logs (#740)
1 parent 473f137 commit 24d4acb

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/databricks/labs/ucx/framework/dashboards.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ def _remote_folder_object(self) -> workspace.ObjectInfo:
192192
def _installed_query_state(self):
193193
if not self._state.dashboards:
194194
for k, v in self._state_pre_v06().items():
195-
prefix, suffix = k.split(":", 2)
195+
prefix, suffix = [*k.split(":"), None][:2]
196196
match suffix:
197197
case "dashboard_id":
198198
self._state.dashboards[prefix] = v

src/databricks/labs/ucx/install.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ def _install_spark_config_for_hms_lineage(self):
201201
gscript = hms_lineage.check_lineage_spark_config_exists()
202202
if gscript:
203203
if gscript.enabled:
204-
logger.info("Already exists and enabled. Skipped creating a new one.")
204+
logger.info("Global Init Script already exists and enabled. Skipped creating a new one.")
205205
elif not gscript.enabled and self._prompts:
206206
if self._prompts.confirm(
207207
"Your Global Init Script with required spark config is disabled, Do you want to enable it?"
@@ -286,6 +286,7 @@ def _create_dashboards(self):
286286
@property
287287
def _warehouse_id(self) -> str:
288288
if self.current_config.warehouse_id is not None:
289+
logger.info("Fetching warehouse_id from a config")
289290
return self.current_config.warehouse_id
290291
warehouses = [_ for _ in self._ws.warehouses.list() if _.warehouse_type == EndpointInfoWarehouseType.PRO]
291292
warehouse_id = self.current_config.warehouse_id
@@ -863,7 +864,7 @@ def _remove_warehouse(self):
863864
try:
864865
warehouse_name = self._ws.warehouses.get(self.current_config.warehouse_id).name
865866
if warehouse_name.startswith(WAREHOUSE_PREFIX):
866-
logger.info("Deleting warehouse_name.")
867+
logger.info(f"Deleting {warehouse_name}.")
867868
self._ws.warehouses.delete(id=self.current_config.warehouse_id)
868869
except InvalidParameterValue:
869870
logger.error("Error accessing warehouse details")

0 commit comments

Comments
 (0)