Installation checks if either configs ("enableProjectTypeInWorkspace", "enableWorkspaceFilesystem") are 'false' and makes them 'true' for valid installation

src/databricks/labs/ucx/install.py

@@ -368,6 +368,7 @@ def folder(self):
 
     def run(self):
         logger.info(f"Installing UCX v{PRODUCT_INFO.version()}")
+        self._enable_files_in_repos()
         Threads.strict(
             "installing components",
             [
@@ -962,6 +963,21 @@ def validate_and_run(self, step: str):
         if not self.validate_step(step):
             self.run_workflow(step)
 
+    def _enable_files_in_repos(self):
+        # check if "enableProjectTypeInWorkspace" and "enableWorkspaceFilesystem" are set to false
+        project_type = self._ws.workspace_conf.get_status("enableProjectTypeInWorkspace")
+        workspace_file_system = self._ws.workspace_conf.get_status("enableWorkspaceFilesystem")
+
+        logger.debug("Checking Files In Repos configuration")
+
+        if project_type["enableProjectTypeInWorkspace"] == "false":
+            logger.debug("enableProjectTypeInWorkspace is False, enabling the config")
+            self._ws.workspace_conf.set_status("enableProjectTypeInWorkspace")
+
+        if workspace_file_system["enableWorkspaceFilesystem"] == "false":
+            logger.debug("enableWorkspaceFilesystem is False, enabling the config")
+            self._ws.workspace_conf.set_status("enableWorkspaceFilesystem")
+
 
 if __name__ == "__main__":
     logger = get_logger(__file__)

tests/integration/test_installation.py

@@ -310,3 +310,12 @@ def test_uninstallation(ws, sql_backend, new_installation):
         ws.jobs.get(job_id=assessment_job_id)
     with pytest.raises(NotFound):
         sql_backend.execute(f"show tables from hive_metastore.{install.config.inventory_database}")
+
+
+def test_files_in_repos_enablement(new_installation):
+    try:
+        install = new_installation()
+        install.uninstall()
+    except NotFound as e:
+        raise AssertionError() from e
+    assert True

tests/unit/test_install.py

@@ -1,4 +1,5 @@
 import json
+import random
 from datetime import timedelta
 from unittest.mock import MagicMock, create_autospec
 
@@ -93,6 +94,14 @@ def ws():
     return ws
 
 
+def mock_get_status(*args, **kwargs):
+    random_return_value = random.choice(["true", "false"])
+    if args[0] == 'enableProjectTypeInWorkspace':
+        return {"enableProjectTypeInWorkspace": random_return_value}
+    elif args[0] == 'enableWorkspaceFilesystem':
+        return {"enableWorkspaceFilesystem": random_return_value}
+
+
 def created_job(ws: MagicMock, name: str):
     for call in ws.jobs.method_calls:
         if call.kwargs['name'] == name:
@@ -1045,3 +1054,27 @@ def test_repair_run_result_state(ws, caplog, mock_installation_with_jobs, any_pr
 
     workspace_installation.repair_run("assessment")
     assert "Please try after sometime" in caplog.text
+
+
+def test_enable_files_in_repos(ws, mock_installation, any_prompt, mocker):
+    sql_backend = MockBackend(
+        fails_on_first={'CREATE TABLE': '[UNRESOLVED_COLUMN.WITH_SUGGESTION] A column, variable is incorrect'}
+    )
+    wheels = create_autospec(WheelsV2)
+
+    ws.workspace_conf.get_status = mocker.patch(
+        "databricks.sdk.service.settings.WorkspaceConfAPI.get_status", side_effect=mock_get_status
+    )
+
+    workspace_installation = WorkspaceInstallation(
+        WorkspaceConfig(inventory_database='ucx'),
+        mock_installation,
+        sql_backend,
+        wheels,
+        ws,
+        any_prompt,
+        timedelta(seconds=1),
+    )
+
+    workspace_installation._enable_files_in_repos()
+    assert True