Skip to content

Fix #4889 - IngestionPipeline reads source connection dynamically; Fix #4888 - Remove force deploy#5228

Merged
pmbrull merged 3 commits into
open-metadata:mainfrom
pmbrull:issue-4889-2
Jun 2, 2022
Merged

Fix #4889 - IngestionPipeline reads source connection dynamically; Fix #4888 - Remove force deploy#5228
pmbrull merged 3 commits into
open-metadata:mainfrom
pmbrull:issue-4889-2

Conversation

@pmbrull

@pmbrull pmbrull commented May 31, 2022

Copy link
Copy Markdown
Member

Describe your changes :

Fix #4889 - IngestionPipeline reads source connection dynamically
Fix #4888 - Remove force deploy

Type of change :

  • Bug fix
  • Improvement
  • New feature
  • [] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [] Documentation

Frontend Preview (Screenshots) :

For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

  • I have read the CONTRIBUTING document.
  • I have commented on my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • All new and existing tests passed.

Reviewers

@pmbrull
pmbrull requested review from a team May 31, 2022 08:28
@github-actions

Copy link
Copy Markdown
Contributor

Schema Change Detected. Needs ingestion-core version bump

Please run make core_bump_version_dev in the project's root and commit the changes to _version.py in this PR. Please ignore if this has been handled already.

@pmbrull pmbrull changed the title Fix #4889 - IngestionPipeline reads source connection dynamically Fix #4889 - IngestionPipeline reads source connection dynamically; Fix #4888 - Remove force deploy May 31, 2022
@sonarqubecloud

Copy link
Copy Markdown

[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

IngestionPipeline gets updated when the service connection changes IngestionPipeline remove forceDeploy

2 participants