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

Improved Purview Integration #190

Merged
merged 8 commits into from
Oct 25, 2021
Merged

Conversation

marvinbuss
Copy link
Collaborator

@marvinbuss marvinbuss commented Oct 15, 2021

This PR fixes
This PR will improve the Purview integration for Data Factory, Synapse and other Services that are deployed as part of the Data Landing Zone. We will automatically create the managed Private Endpoints on the Managed Virtual Network to push Lineage metadata from Data Factory and Synapse into Purview.
This PR will also use the new Bicep release for the ARM build process.

@marvinbuss marvinbuss added the enhancement New feature or request label Oct 15, 2021
@marvinbuss marvinbuss self-assigned this Oct 15, 2021
Copy link
Collaborator Author

@marvinbuss marvinbuss left a comment

Choose a reason for hiding this comment

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

LGTM!

@marvinbuss marvinbuss added this to the v1.2.0 milestone Oct 18, 2021
@marvinbuss marvinbuss marked this pull request as ready for review October 18, 2021 10:27
@marvinbuss marvinbuss requested a review from a team as a code owner October 18, 2021 10:27
@amanjeetsingh
Copy link
Contributor

This is under review now. Thanks!

@amanjeetsingh amanjeetsingh added the documentation Improvements or additions to documentation label Oct 19, 2021
amanjeetsingh
amanjeetsingh previously approved these changes Oct 19, 2021
Copy link
Contributor

@amanjeetsingh amanjeetsingh left a comment

Choose a reason for hiding this comment

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

Good to go.

Copy link
Contributor

@amanjeetsingh amanjeetsingh left a comment

Choose a reason for hiding this comment

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

Reviewed additional changes - approved.

@marvinbuss marvinbuss merged commit 94f89cb into main Oct 25, 2021
@marvinbuss marvinbuss deleted the marvinbuss/Purview_integration branch October 25, 2021 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
committed Committed item documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Private Link Connectivity between Data Factory and Purview
2 participants