From 80a4399c1a65ed101a9264a1dd8b46faff39582c Mon Sep 17 00:00:00 2001
From: Sriharsha Chintalapani <harshach@users.noreply.github.com>
Date: Wed, 30 Nov 2022 13:09:57 -0800
Subject: [PATCH] add required fields for amundsen connection (#9039)

* add required fields for amundsen connection

* Remove unused modelClass field

Co-authored-by: ulixius9 <mayursingal9@gmail.com>
---
 .../content/connectors/metadata/amundsen/index.md            | 2 --
 .../services/connections/metadata/amundsenConnection.md      | 1 -
 .../services/connections/metadata/amundsenConnection.json    | 5 +----
 3 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/openmetadata-docs/content/connectors/metadata/amundsen/index.md b/openmetadata-docs/content/connectors/metadata/amundsen/index.md
index deb34beb0b1..6708121cfdf 100644
--- a/openmetadata-docs/content/connectors/metadata/amundsen/index.md
+++ b/openmetadata-docs/content/connectors/metadata/amundsen/index.md
@@ -81,7 +81,6 @@ source:
       maxConnectionLifeTime: <time in secs.>
       validateSSL: <true or false>
       encrypted: <true or false>
-      modelClass: <modelclass>
   sourceConfig:
     config: {}
 
@@ -104,7 +103,6 @@ You can find all the definitions and types for the `serviceConnection` [here](ht
 - `maxConnectionLifeTime` (optional): Maximum connection lifetime for the Amundsen Neo4j Connection 
 - `validateSSL` (optional): Enable SSL validation for the Amundsen Neo4j Connection. 
 - `encrypted` (Optional): Enable encryption for the Amundsen Neo4j Connection. 
-- `modelClass` (Optional): Model Class for the Amundsen Neo4j Connection.
 
 ### Sink Configuration
 
diff --git a/openmetadata-docs/content/main-concepts/metadata-standard/schemas/entity/services/connections/metadata/amundsenConnection.md b/openmetadata-docs/content/main-concepts/metadata-standard/schemas/entity/services/connections/metadata/amundsenConnection.md
index 2683702d5ff..0116fc8537b 100644
--- a/openmetadata-docs/content/main-concepts/metadata-standard/schemas/entity/services/connections/metadata/amundsenConnection.md
+++ b/openmetadata-docs/content/main-concepts/metadata-standard/schemas/entity/services/connections/metadata/amundsenConnection.md
@@ -16,7 +16,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/metad
 - **`maxConnectionLifeTime`** *(integer)*: Maximum connection lifetime for the Amundsen Neo4j Connection. Default: `50`.
 - **`validateSSL`** *(boolean)*: Enable SSL validation for the Amundsen Neo4j Connection. Default: `false`.
 - **`encrypted`** *(boolean)*: Enable encryption for the Amundsen Neo4j Connection. Default: `false`.
-- **`modelClass`** *(string)*: Model Class for the Amundsen Neo4j Connection.
 - **`supportsMetadataExtraction`**: Refer to *../connectionBasicType.json#/definitions/supportsMetadataExtraction*.
 ## Definitions
 
diff --git a/openmetadata-spec/src/main/resources/json/schema/entity/services/connections/metadata/amundsenConnection.json b/openmetadata-spec/src/main/resources/json/schema/entity/services/connections/metadata/amundsenConnection.json
index b65b33cd723..2101cf80041 100644
--- a/openmetadata-spec/src/main/resources/json/schema/entity/services/connections/metadata/amundsenConnection.json
+++ b/openmetadata-spec/src/main/resources/json/schema/entity/services/connections/metadata/amundsenConnection.json
@@ -50,13 +50,10 @@
       "type": "boolean",
       "default": "false"
     },
-    "modelClass": {
-      "description": "Model Class for the Amundsen Neo4j Connection.",
-      "type": "string"
-    },
     "supportsMetadataExtraction": {
       "$ref": "../connectionBasicType.json#/definitions/supportsMetadataExtraction"
     }
   },
+  "required": ["username", "password", "hostPort"],
   "additionalProperties": false
 }