From 425efd3d7f5d7de6e107747683eedb755d4c77b2 Mon Sep 17 00:00:00 2001 From: Liran Bareket Date: Fri, 21 Feb 2025 13:00:53 -0500 Subject: [PATCH 1/2] Added a message to assign-metastore command. --- src/databricks/labs/ucx/account/metastores.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/databricks/labs/ucx/account/metastores.py b/src/databricks/labs/ucx/account/metastores.py index 94c5b9e2fe..ff129b8605 100644 --- a/src/databricks/labs/ucx/account/metastores.py +++ b/src/databricks/labs/ucx/account/metastores.py @@ -48,6 +48,8 @@ def assign_metastore( # set the default catalog using the default_namespace setting API if default_catalog is not None: self._set_default_catalog(workspace_id, default_catalog) + logger.info(f"Metastore {metastore_id} assigned to workspace {workspace_id}") + logger.info("Run the 'create-ucx-catalog' command to create the UCX catalog in case it does not exist") def _get_region(self, workspace_id: int) -> str | None: workspace = self._ac.workspaces.get(workspace_id) From 556fea56effbf64cf45a5ec65953df9deabc2412 Mon Sep 17 00:00:00 2001 From: Liran Bareket Date: Mon, 24 Feb 2025 11:26:01 -0500 Subject: [PATCH 2/2] Addressed review comments --- src/databricks/labs/ucx/account/metastores.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/databricks/labs/ucx/account/metastores.py b/src/databricks/labs/ucx/account/metastores.py index ff129b8605..44e60bf0d3 100644 --- a/src/databricks/labs/ucx/account/metastores.py +++ b/src/databricks/labs/ucx/account/metastores.py @@ -45,11 +45,11 @@ def assign_metastore( else: metastore_id = list(metastore_choices.values())[0] self._ac.metastore_assignments.create(workspace_id, metastore_id) + logger.info(f"Metastore {metastore_id} assigned to workspace {workspace_id}") # set the default catalog using the default_namespace setting API if default_catalog is not None: self._set_default_catalog(workspace_id, default_catalog) - logger.info(f"Metastore {metastore_id} assigned to workspace {workspace_id}") - logger.info("Run the 'create-ucx-catalog' command to create the UCX catalog in case it does not exist") + logger.info("Run the 'create-ucx-catalog' command to create the UCX catalog for tracking your migration progress") def _get_region(self, workspace_id: int) -> str | None: workspace = self._ac.workspaces.get(workspace_id)