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

Create Azure ML RBAC roles #2539

Merged
merged 8 commits into from
Sep 7, 2022
Merged

Conversation

marrobi
Copy link
Member

@marrobi marrobi commented Sep 3, 2022

Resolves #1204

What is being addressed

Add principles assigned a role in the workpsace as data scientists.

How is this addressed

  • Add script that gets principal IDs (users and groups) assigned app roles
  • Add AAD creds to porter
  • Assign AzureML Data Scientist Roles to principals
  • Update documentation
  • Update CHANGELOG.md
  • Filter to workspace researchers

@github-actions
Copy link

github-actions bot commented Sep 3, 2022

Unit Test Results

0 tests   - 2   0 ✔️  - 1   0s ⏱️ - 1h 9m 35s
0 suites  - 1   0 💤 ±0 
0 files    - 1   0  - 1 

Results for commit 99675e4. ± Comparison against base commit f707a1d.

♻️ This comment has been updated with latest results.

@marrobi marrobi self-assigned this Sep 3, 2022
@marrobi marrobi changed the title Create AML RBAC roles Create Azure ML RBAC roles Sep 3, 2022
@marrobi marrobi marked this pull request as ready for review September 5, 2022 15:12
@marrobi
Copy link
Member Author

marrobi commented Sep 5, 2022

/test-force-approve role assignments are not tested

@github-actions
Copy link

github-actions bot commented Sep 5, 2022

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 44e240d)

(in response to this comment from @marrobi)

@marrobi
Copy link
Member Author

marrobi commented Sep 7, 2022

/test-force-approve role assignments are not tested

@marrobi marrobi enabled auto-merge (squash) September 7, 2022 11:17
@github-actions
Copy link

github-actions bot commented Sep 7, 2022

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 99675e4)

(in response to this comment from @marrobi)

@marrobi marrobi merged commit 5f1d801 into microsoft:main Sep 7, 2022
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.

Assign users Azure ML and prevent the deployment of compute other than via TRE API
2 participants