Merge branch 'main' into fix-iceberg-rest-catalog-config

ingestion/src/_openmetadata_testutils/ometa.py

@@ -11,12 +11,14 @@
 OM_JWT = "eyJraWQiOiJHYjM4OWEtOWY3Ni1nZGpzLWE5MmotMDI0MmJrOTQzNTYiLCJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiJ9.eyJzdWIiOiJhZG1pbiIsImlzQm90IjpmYWxzZSwiaXNzIjoib3Blbi1tZXRhZGF0YS5vcmciLCJpYXQiOjE2NjM5Mzg0NjIsImVtYWlsIjoiYWRtaW5Ab3Blbm1ldGFkYXRhLm9yZyJ9.tS8um_5DKu7HgzGBzS1VTA5uUjKWOCU0B_j08WXBiEC0mr0zNREkqVfwFDD-d24HlNEbrqioLsBuFRiwIWKc1m_ZlVQbG7P36RUxhuv2vbSp80FKyNM-Tj93FDzq91jsyNmsQhyNv_fNr3TXfzzSPjHt8Go0FMMP66weoKMgW2PbXlhVKwEuXUHyakLLzewm9UMeQaEiRzhiTMU3UkLXcKbYEJJvfNFcLwSl9W8JCO_l0Yj3ud-qt_nQYEZwqW6u5nfdQllN133iikV4fM5QZsMCnm8Rq1mvLR0y9bmJiD7fwM1tmJ791TUWqmKaTnP49U493VanKpUAfzIiOiIbhg"
 
 
-def int_admin_ometa(url: str = "http://localhost:8585/api") -> OpenMetadata:
+def int_admin_ometa(
+    url: str = "http://localhost:8585/api", jwt: str = OM_JWT
+) -> OpenMetadata:
     """Initialize the ometa connection with default admin:admin creds"""
     server_config = OpenMetadataConnection(
         hostPort=url,
         authProvider=AuthProvider.openmetadata,
-        securityConfig=OpenMetadataJWTClientConfig(jwtToken=CustomSecretStr(OM_JWT)),
+        securityConfig=OpenMetadataJWTClientConfig(jwtToken=CustomSecretStr(jwt)),
     )
     metadata = OpenMetadata(server_config)
     assert metadata.health_check()

ingestion/src/metadata/ingestion/ometa/mixins/suggestions_mixin.py

@@ -13,8 +13,13 @@
 
 To be used by OpenMetadata class
 """
-from metadata.generated.schema.entity.feed.suggestion import Suggestion
+from typing import Union
+
+from metadata.generated.schema.entity.feed.suggestion import Suggestion, SuggestionType
+from metadata.generated.schema.type import basic
+from metadata.generated.schema.type.basic import FullyQualifiedEntityName
 from metadata.ingestion.ometa.client import REST
+from metadata.ingestion.ometa.utils import model_str
 from metadata.utils.logger import ometa_logger
 
 logger = ometa_logger()
@@ -30,12 +35,50 @@ class OMetaSuggestionsMixin:
     client: REST
 
     def update_suggestion(self, suggestion: Suggestion) -> Suggestion:
-        """
-        Update an existing Suggestion with new fields
-        """
+        """Update an existing Suggestion with new fields"""
         resp = self.client.put(
             f"{self.get_suffix(Suggestion)}/{str(suggestion.root.id.root)}",
             data=suggestion.model_dump_json(),
         )
 
         return Suggestion(**resp)
+
+    def accept_suggestion(self, suggestion_id: Union[str, basic.Uuid]) -> None:
+        """Accept a given suggestion"""
+        self.client.put(
+            f"{self.get_suffix(Suggestion)}/{model_str(suggestion_id)}/accept",
+        )
+
+    def reject_suggestion(self, suggestion_id: Union[str, basic.Uuid]) -> None:
+        """Reject a given suggestion"""
+        self.client.put(
+            f"{self.get_suffix(Suggestion)}/{model_str(suggestion_id)}/reject",
+        )
+
+    def accept_all_suggestions(
+        self,
+        fqn: Union[str, FullyQualifiedEntityName],
+        user_id: Union[str, basic.Uuid],
+        suggestion_type: SuggestionType = SuggestionType.SuggestDescription,
+    ) -> None:
+        """Accept all suggestions"""
+        self.client.put(
+            f"{self.get_suffix(Suggestion)}/accept-all?"
+            f"userId={model_str(user_id)}&"
+            f"entityFQN={model_str(fqn)}&"
+            f"suggestionType={suggestion_type.value}",
+        )
+
+    def reject_all_suggestions(
+        self,
+        fqn: Union[str, FullyQualifiedEntityName],
+        user_id: Union[str, basic.Uuid],
+        suggestion_type: SuggestionType = SuggestionType.SuggestDescription,
+    ) -> None:
+        """Accept all suggestions"""
+        self.client.put(
+            f"{self.get_suffix(Suggestion)}/reject-all?"
+            f"userId={model_str(user_id)}&"
+            f"entityFQN={model_str(fqn)}&"
+            f"suggestionType={suggestion_type.value}",
+        )

ingestion/src/metadata/ingestion/ometa/ometa_api.py

@@ -19,6 +19,7 @@
 
 from pydantic import BaseModel
 
+from metadata.generated.schema.api.createBot import CreateBot
 from metadata.generated.schema.api.services.ingestionPipelines.createIngestionPipeline import (
     CreateIngestionPipelineRequest,
 )
@@ -172,13 +173,16 @@ def get_suffix(entity: Type[T]) -> str:
 
         return route
 
-    def get_module_path(self, entity: Type[T]) -> str:
+    def get_module_path(self, entity: Type[T]) -> Optional[str]:
         """
         Based on the entity, return the module path
         it is found inside generated
         """
         if issubclass(entity, CreateIngestionPipelineRequest):
             return "services.ingestionPipelines"
+        if issubclass(entity, CreateBot):
+            # Bots schemas don't live inside any subdirectory
+            return None
         return entity.__module__.split(".")[-2]
 
     def get_create_entity_type(self, entity: Type[T]) -> Type[C]:

ingestion/src/metadata/ingestion/ometa/routes.py

@@ -21,6 +21,7 @@
     CreateClassificationRequest,
 )
 from metadata.generated.schema.api.classification.createTag import CreateTagRequest
+from metadata.generated.schema.api.createBot import CreateBot
 from metadata.generated.schema.api.data.createAPICollection import (
     CreateAPICollectionRequest,
 )
@@ -213,7 +214,8 @@
     User.__name__: "/users",
     CreateUserRequest.__name__: "/users",
     AuthenticationMechanism.__name__: "/users/auth-mechanism",
-    Bot.__name__: "/bots",  # We won't allow bot creation from the client
+    Bot.__name__: "/bots",
+    CreateBot.__name__: "/bots",
     # Roles
     Role.__name__: "/roles",
     CreateRoleRequest.__name__: "/roles",

ingestion/tests/integration/ometa/test_ometa_suggestion_api.py

@@ -14,14 +14,20 @@
 """
 from unittest import TestCase
 
+import pytest
+
 from _openmetadata_testutils.ometa import int_admin_ometa
+from metadata.generated.schema.api.createBot import CreateBot
 from metadata.generated.schema.api.feed.createSuggestion import CreateSuggestionRequest
+from metadata.generated.schema.api.teams.createUser import CreateUserRequest
+from metadata.generated.schema.auth.jwtAuth import JWTAuthMechanism, JWTTokenExpiry
+from metadata.generated.schema.entity.bot import Bot
 from metadata.generated.schema.entity.data.database import Database
 from metadata.generated.schema.entity.data.databaseSchema import DatabaseSchema
 from metadata.generated.schema.entity.data.table import Table
 from metadata.generated.schema.entity.feed.suggestion import Suggestion, SuggestionType
 from metadata.generated.schema.entity.services.databaseService import DatabaseService
-from metadata.generated.schema.entity.teams.user import User
+from metadata.generated.schema.entity.teams.user import AuthenticationMechanism, User
 from metadata.generated.schema.type.basic import EntityLink
 from metadata.generated.schema.type.tagLabel import (
     LabelType,
@@ -30,11 +36,40 @@
     TagLabel,
     TagSource,
 )
+from metadata.ingestion.ometa.client import APIError
+from metadata.ingestion.ometa.ometa_api import OpenMetadata
+from metadata.ingestion.source.database.clickhouse.utils import Tuple
 from metadata.utils.entity_link import get_entity_link
 
 from ..integration_base import generate_name, get_create_entity, get_create_service
 
 
+def _create_bot(metadata: OpenMetadata) -> Tuple[User, Bot]:
+    """Create a bot"""
+    bot_name = generate_name()
+    user: User = metadata.create_or_update(
+        data=CreateUserRequest(
+            name=bot_name,
+            email=f"{bot_name.root}@user.com",
+            isBot=True,
+            authenticationMechanism=AuthenticationMechanism(
+                authType="JWT",
+                config=JWTAuthMechanism(
+                    JWTTokenExpiry=JWTTokenExpiry.Unlimited,
+                ),
+            ),
+        )
+    )
+    bot: Bot = metadata.create_or_update(
+        data=CreateBot(
+            name=bot_name,
+            botUser=bot_name.root,
+        )
+    )
+
+    return user, bot
+
+
 class OMetaSuggestionTest(TestCase):
     """
     Run this integration test with the local API available
@@ -109,6 +144,138 @@ def test_create_description_suggestion(self):
         # Suggestions only support POST (not PUT)
         self.metadata.create(suggestion_request)
 
+    def test_accept_reject_suggestion(self):
+        """We can create and accept a suggestion"""
+        suggestion_request = CreateSuggestionRequest(
+            description="i won't be accepted",
+            type=SuggestionType.SuggestDescription,
+            entityLink=EntityLink(
+                root=get_entity_link(Table, fqn=self.table.fullyQualifiedName.root)
+            ),
+        )
+
+        self.metadata.patch_description(
+            entity=Table,
+            source=self.metadata.get_by_name(
+                entity=Table, fqn=self.table.fullyQualifiedName.root
+            ),
+            description="I come from a patch",
+        )
+
+        # Suggestions only support POST (not PUT)
+        suggestion = self.metadata.create(suggestion_request)
+
+        # We can reject a suggestion
+        self.metadata.reject_suggestion(suggestion.root.id)
+        updated_table: Table = self.metadata.get_by_name(
+            entity=Table, fqn=self.table.fullyQualifiedName.root
+        )
+        assert updated_table.description.root == "I come from a patch"
+
+        # We create a new suggestion and accept it this time
+        suggestion_request = CreateSuggestionRequest(
+            description="something new",
+            type=SuggestionType.SuggestDescription,
+            entityLink=EntityLink(
+                root=get_entity_link(Table, fqn=self.table.fullyQualifiedName.root)
+            ),
+        )
+
+        # Suggestions only support POST (not PUT)
+        suggestion = self.metadata.create(suggestion_request)
+
+        # We can accept a suggestion
+        self.metadata.accept_suggestion(suggestion.root.id)
+        updated_table: Table = self.metadata.get_by_name(
+            entity=Table, fqn=self.table.fullyQualifiedName.root
+        )
+        assert updated_table.description.root == "something new"
+
+    def test_accept_suggest_delete_user(self):
+        """We can accept the suggestion of a deleted user"""
+
+        user, bot = _create_bot(self.metadata)
+        bot_metadata = int_admin_ometa(
+            jwt=user.authenticationMechanism.config.JWTToken.get_secret_value()
+        )
+
+        # We create a new suggestion and accept it this time
+        suggestion_request = CreateSuggestionRequest(
+            description="something new",
+            type=SuggestionType.SuggestDescription,
+            entityLink=EntityLink(
+                root=get_entity_link(Table, fqn=self.table.fullyQualifiedName.root)
+            ),
+        )
+
+        # Suggestions only support POST (not PUT)
+        suggestion = bot_metadata.create(suggestion_request)
+        assert suggestion
+
+        # Delete the bot
+        self.metadata.delete(
+            entity=Bot,
+            entity_id=bot.id,
+            recursive=True,
+            hard_delete=True,
+        )
+
+        # We won't find the suggestion
+        with pytest.raises(APIError) as exc:
+            self.metadata.accept_suggestion(suggestion.root.id)
+
+        assert (
+            str(exc.value)
+            == f"Suggestion instance for {suggestion.root.id.root} not found"
+        )
+
+    def test_accept_all_delete_user(self):
+        """We can accept all suggestions of a deleted user"""
+        user, bot = _create_bot(self.metadata)
+        bot_metadata = int_admin_ometa(
+            jwt=user.authenticationMechanism.config.JWTToken.get_secret_value()
+        )
+
+        self.metadata.patch_description(
+            entity=Table,
+            source=self.metadata.get_by_name(
+                entity=Table, fqn=self.table.fullyQualifiedName.root
+            ),
+            description="I come from a patch",
+        )
+
+        # We create a new suggestion and accept it this time
+        suggestion_request = CreateSuggestionRequest(
+            description="something new from test_accept_all_delete_user",
+            type=SuggestionType.SuggestDescription,
+            entityLink=EntityLink(
+                root=get_entity_link(Table, fqn=self.table.fullyQualifiedName.root)
+            ),
+        )
+
+        # Suggestions only support POST (not PUT)
+        suggestion = bot_metadata.create(suggestion_request)
+        assert suggestion
+
+        # Delete the bot
+        self.metadata.delete(
+            entity=Bot,
+            entity_id=bot.id,
+            recursive=True,
+            hard_delete=True,
+        )
+
+        # This will do nothing, since there's no suggestions there
+        self.metadata.accept_all_suggestions(
+            fqn=self.table.fullyQualifiedName.root,
+            user_id=user.id,
+            suggestion_type=SuggestionType.SuggestDescription,
+        )
+        updated_table: Table = self.metadata.get_by_name(
+            entity=Table, fqn=self.table.fullyQualifiedName.root
+        )
+        assert updated_table.description.root == "I come from a patch"
+
     def test_create_tag_suggestion(self):
         """We can create a suggestion"""
         suggestion_request = CreateSuggestionRequest(

openmetadata-docs/content/v1.4.x/connectors/metadata/alation/index.md

@@ -1,6 +1,7 @@
 ---
 title: Alation
 slug: /connectors/metadata/alation
+collate: true
 ---
 
 {% connectorDetailsHeader

openmetadata-docs/content/v1.4.x/connectors/metadata/alation/yaml.md

@@ -1,6 +1,7 @@
 ---
 title: Run the Alation Connector Externally
 slug: /connectors/metadata/alation/yaml
+collate: true
 ---
 
 {% connectorDetailsHeader

openmetadata-docs/content/v1.4.x/connectors/storage/adls/index.md

@@ -1,6 +1,7 @@
 ---
 title: ADLS
 slug: /connectors/storage/adls
+collate: true
 ---
 
 {% connectorDetailsHeader

openmetadata-docs/content/v1.4.x/connectors/storage/adls/yaml.md

@@ -1,6 +1,7 @@
 ---
 title: Run the ADLS Connector Externally
 slug: /connectors/storage/adls/yaml
+collate: true
 ---
 
 {% connectorDetailsHeader

openmetadata-docs/content/v1.5.x/connectors/database/synapse/index.md

@@ -1,6 +1,7 @@
 ---
 title: Synapse
 slug: /connectors/database/synapse
+collate: true
 ---
 
 {% connectorDetailsHeader

openmetadata-docs/content/v1.5.x/connectors/database/synapse/troubleshooting.md

@@ -1,6 +1,7 @@
 ---
 title: Synapse Connector Troubleshooting
 slug: /connectors/database/synapse/troubleshooting
+collate: true
 ---
 
 # Troubleshooting

openmetadata-docs/content/v1.5.x/connectors/database/synapse/yaml.md

@@ -1,6 +1,7 @@
 ---
 title: Run the Synapse Connector Externally
 slug: /connectors/database/synapse/yaml
+collate: true
 ---
 
 {% connectorDetailsHeader

openmetadata-docs/content/v1.5.x/connectors/metadata/alation/index.md

@@ -1,6 +1,7 @@
 ---
 title: Alation
 slug: /connectors/metadata/alation
+collate: true
 ---
 
 {% connectorDetailsHeader