-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Update service principal role to AzureML Compute Operator for improved security #2169
Conversation
…d security Signed-off-by: miguelgfierro <[email protected]>
Issue #2170 |
We added the reader role to the SP and tried again to run the tests: https://github.com/recommenders-team/recommenders/actions/runs/10996608019 We got a new error:
We tried to also add AzureML Data Scientist. Testing: https://github.com/recommenders-team/recommenders/actions/runs/10996798072/job/30530817759 |
Signed-off-by: miguelgfierro <[email protected]>
Some documentation on the roles https://learn.microsoft.com/en-us/azure/machine-learning/how-to-assign-roles?view=azureml-api-2&tabs=team-lead#default-roles |
Everything worked. Merging. |
Description
Related Issues
References
Checklist:
git commit -s -m "your commit message"
.staging branch
AND NOT TOmain branch
.