Skip to content
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

Merged
merged 13 commits into from
Oct 13, 2022
Merged

OpenMetadata install script #1731

merged 13 commits into from
Oct 13, 2022

Conversation

cdoron
Copy link
Collaborator

@cdoron cdoron commented Sep 19, 2022

No description provided.

@cdoron cdoron requested a review from roytman September 19, 2022 14:52
@@ -0,0 +1,17 @@
// Copyright 2020 IBM Corp.
Copy link
Collaborator

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 ?

Copy link
Collaborator Author

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.

Copy link
Collaborator Author

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",
Copy link
Collaborator

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

Copy link
Collaborator Author

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)
Copy link
Collaborator

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

@cdoron cdoron changed the title Om install script OpenMetadata install script Oct 12, 2022
export OPENMETADATA_HELM_CHART_VERSION=0.0.39

export FYBRIK_BRANCH="${FYBRIK_BRANCH:-master}"
export FYBRIK_GITHUB_ORGANIZATION="${FYBRIK_NAMESPACE:-fybrik}"
Copy link
Collaborator

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}

@roytman roytman enabled auto-merge (squash) October 13, 2022 14:29
@roytman roytman merged commit 0378a91 into fybrik:master Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants