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.

Sign up for GitHub

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

Jump to bottom

Added Dataverse Integration Template #19

Open
wants to merge 32 commits into
base: main
Choose a base branch
from
Open

Added Dataverse Integration Template #19

wants to merge 32 commits into from

Conversation

marvinbuss
Copy link
Contributor

@marvinbuss marvinbuss commented Sep 16, 2021 •
edited
Loading

This PR fixes

  • Bicep templates + Built ARM templates
  • Sample parameter file
  • Add Powershell to automate the connection from Power App to Synapse
  • Testing
  • Sample Deploy to Azure Button
  • README for solution pattern

Open Questions:

  • Where should we add this in the folder structure, as it is relevant for all industry solutions?
  • What are we going to do about the region specific API endpoints?

@marvinbuss marvinbuss added the enhancement New feature or request label Sep 16, 2021
@marvinbuss marvinbuss self-assigned this Sep 16, 2021
@marvinbuss
Copy link
Contributor Author

This solution requires access to storage account keys as well as quite a number of access rights for EAs. Due to this we will most likely have to add another external storage account as intermediate layer before landing this data onto the core Datalake.

@marvinbuss
* Added Role Assignments to ARM/Bicep for PP
f1d1f40 
* Added pwsh scripts for automating the setup of synch
* Added role assignment templates
* Updated ARM
@marvinbuss
* Added missing parameters
019d25a 
* Removed old role assignments
* Removed Container Creation in Script
@marvinbuss
update linting
2e41c7f
@marvinbuss
removing unnecessary parameter
e8686f9
@marvinbuss
Fixed bugs
219f04b
@marvinbuss
fix spark pool bug
2b3bb85
@marvinbuss
add outputs for seamless deployment across clouds
13e2c5c
@marvinbuss
* Added additional logging for troubleshooting
9de0afa 
* Added function for creating file systems
* fixed bugs in workflow
* Generate Access Token with correct scope
* Added some sleep in between actions because of backend processes
@marvinbuss
Update docs
455a42b
@marvinbuss
linting
13a4244
@marvinbuss
Updated to reference implementation
4261f1a
@marvinbuss
Updated private dns
b234764
@marvinbuss
added portal exp
0fde020
@marvinbuss
updated header
dae400d
@marvinbuss
Added Portal UI
d33e45b
@marvinbuss marvinbuss requested a review from krnese September 30, 2021 18:01
@marvinbuss marvinbuss marked this pull request as ready for review September 30, 2021 18:09
@marvinbuss
Copy link
Contributor Author

Open Questions:

  • Where should we add this in the folder structure, as it is relevant for all industry solutions?
  • What are we going to do about the region specific API endpoints?

@marvinbuss marvinbuss requested review from a team and removed request for krnese October 4, 2021 09:25
@marvinbuss marvinbuss added the documentation Improvements or additions to documentation label Oct 5, 2021
@marvinbuss marvinbuss requested a review from krnese October 6, 2021 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krnese krnese Awaiting requested review from krnese

At least 1 approving review is required to merge this pull request.

Assignees

@marvinbuss marvinbuss

Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@marvinbuss @krnese