-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix #10344 - Use ingestionPipeline retries when deploying #12926
Conversation
Passing run #28269 ↗︎Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
[open-metadata-ui] Kudos, SonarCloud Quality Gate passed!
|
[open-metadata-airflow-apis] Kudos, SonarCloud Quality Gate passed!
|
[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed! |
Describe your changes:
Fixes #10344
When creating the ingestion, we should allow users to configure the number of
retries
if the workflow fails. By default, it will be0
.We need to pass this information in the
ingestionPipeline.airflowConfig.retries
field. We have already updated the ingestion framework to use it.We'll do the changes in the Collate repo accordingly
Screen.Recording.2023-08-21.at.5.54.05.PM.mov
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>