Handle exceptions with no error_code attribute while crawling permissions (#2079)

Avoid `AttributeError` error when exceptions without an `error_code`
attribute are caught during the assessment job crawling permissions.

### Linked issues
Resolves #2078 

### Tests

- [ ] manually tested
- [ x] added unit tests
- [ ] added integration tests
- [ ] verified on staging environment (screenshot attached)

Author: qziyuan

src/databricks/labs/ucx/workspace_access/manager.py

@@ -32,11 +32,11 @@ def inventorize_permissions(self):
         items, errors = Threads.gather("crawl permissions", crawler_tasks)
         acute_errors = []
         for error in errors:
-            if error.error_code not in self.ERRORS_TO_IGNORE:
-                logger.error(f"Error while crawling permissions: {error}")
-                acute_errors.append(error)
+            if hasattr(error, 'error_code') and error.error_code in self.ERRORS_TO_IGNORE:
+                logger.info(f"Error while crawling permissions: {error}. Skipping")
                 continue
-            logger.info(f"Error while crawling permissions: {error}. Skipping")
+            logger.error(f"Error while crawling permissions: {error}")
+            acute_errors.append(error)
         if len(acute_errors) > 0:
             raise ManyError(acute_errors)
         logger.info(f"Total crawled permissions: {len(items)}")

tests/unit/workspace_access/test_manager.py

@@ -8,6 +8,7 @@
 from databricks.sdk.errors import DatabricksError
 from databricks.sdk.service import iam
 
+from databricks.labs.blueprint.parallel import ManyError
 from databricks.labs.ucx.workspace_access.base import AclSupport
 from databricks.labs.ucx.workspace_access.groups import MigratedGroup, MigrationState
 from databricks.labs.ucx.workspace_access.manager import PermissionManager, Permissions
@@ -107,6 +108,25 @@ def raise_error():
     )
 
 
+def test_manager_inventorize_fail_with_error(mock_backend, mocker):
+    def raise_error():
+        raise DatabricksError(
+            "Fail the job",
+            error_code="NO_SKIP",
+        )
+
+    def raise_error_no_code():
+        raise TimeoutError
+
+    some_crawler = mocker.Mock()
+    some_crawler.get_crawler_tasks = lambda: [lambda: Permissions("a", "b", "c"), raise_error, raise_error_no_code]
+    permission_manager = PermissionManager(mock_backend, "test_database", [some_crawler])
+
+    with pytest.raises(ManyError) as expected_err:
+        permission_manager.inventorize_permissions()
+    assert len(expected_err.value.errs) == 2
+
+
 def test_manager_apply(mocker):
     sql_backend = MockBackend(
         rows={