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

Assign Azure Reader role to Azure Purview MSI at Data Landing Zone subscription #115

Closed
zeinab-mk opened this issue May 7, 2021 · 3 comments
Labels
backlog Backlog item feature Feature Requests

Comments

@zeinab-mk
Copy link

When Data Landing Zone is deployed, it is nice to have Azure RBAC Reader role automatically assigned to Azure Purview MSI at subscription level. This step is required so customers can register and scan data sources in Purview.

@marvinbuss marvinbuss transferred this issue from another repository May 7, 2021
@marvinbuss
Copy link
Collaborator

marvinbuss commented May 7, 2021

@zeinab-mk Thanks for submitting this!

@marvinbuss marvinbuss added the feature Feature Requests label May 7, 2021
@marvinbuss
Copy link
Collaborator

This is also related to #114. Both items will be closed, once we add a role assignment to the subscription. Optimally. This should even be done at the Management group layer, to capture all data sources within the tenant. But this is out of our scope, which is why we will move forward with the data landing zone subscription scope for now.

@marvinbuss
Copy link
Collaborator

We will not add this for now, since SHIR and Service Principal are required anyways for scans, if all services are behind private endpoints (e.g. Purview, Synapse, Data Factory, etc.). Therefore, we will hold off for now, since this is not something that is actually required when using private endpoints end-to-end.

@marvinbuss marvinbuss removed their assignment Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Backlog item feature Feature Requests
Projects
None yet
Development

No branches or pull requests

3 participants