-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OpenMetadata install script #1731
Conversation
Signed-off-by: Doron Chen <[email protected]>
Signed-off-by: Doron Chen <[email protected]>
Signed-off-by: Doron Chen <[email protected]>
Signed-off-by: Doron Chen <[email protected]>
@@ -0,0 +1,17 @@ | |||
// Copyright 2020 IBM Corp. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you need this start connector here ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The NewOpenMetadataAPIService() method runs a prepareOpenMetadataForFybrik() method which creates the custom properties that we need.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed the go code.
The preparation is now done using the same executable as that of the openmetadata-connector (with different parameters).
|
||
func main() { | ||
conf := map[interface{}]interface{}{ | ||
"openmetadata_endpoint": "http://localhost:8585/api", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can it be an argument or env parameter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The preparation code gets the endpoint, username, and password from an environment variable.
"openmetadata_endpoint": "http://localhost:8585/api", | ||
} | ||
logger := logging.LogInit(logging.CONNECTOR, "OpenMetadata Prepare for Fybrik") | ||
openapi_connector_core.NewOpenMetadataAPIService(conf, &logger) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we separate the connection and tag creation from NewOpenMetadataAPI Service creation, and get the custom properties and tags from external files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
Signed-off-by: Doron Chen <[email protected]>
Signed-off-by: Doron Chen <[email protected]>
Signed-off-by: Doron Chen <[email protected]>
export OPENMETADATA_HELM_CHART_VERSION=0.0.39 | ||
|
||
export FYBRIK_BRANCH="${FYBRIK_BRANCH:-master}" | ||
export FYBRIK_GITHUB_ORGANIZATION="${FYBRIK_NAMESPACE:-fybrik}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
${FYBRIK_NAMESPACE:-fybrik} -> ${FYBRIK_GITHUB_ORGANIZATION:-fybrik}
Signed-off-by: Doron Chen <[email protected]>
No description provided.