Fix missing identity role assignment #14
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
ckittel
commented
Oct 16, 2024
ckittel
commented
- There were two missing role assignments for the managed online endpoint, added those
- Also started to figure out what is going wrong with a telemetry error. I'm running into the same issue as [BUG] Running Prompt flow locally produces errors microsoft/promptflow#3751 -- in the managed online endpoint.
ckittel
added a commit
that referenced
this pull request
Oct 21, 2024
* wip * Move to system managed identities * Remove AML-specific items add missing role assignment (#10) * Remove AML-specific items add missing role assignment * Update infra-as-code/bicep/machinelearning.bicep * deploy user object id (#11) * deploy user object id * Apply suggestions from code review --------- Co-authored-by: Chad Kittel <[email protected]> * Fix user permission (#12) * Remove AML-specific items add missing role assignment * Fix typo * Fix missing \ * remove space * Fix auth issue and add endpoint deployment (#13) * Remove AML-specific items add missing role assignment * Fix typo * Fix missing \ * remove space * Fix permissions and add endpoint * Fix missing identities (#14) * More progress (#15) * Get web app working (#16) * More progress * Get webapp working * Update our TODOs * naming changes (#18) * purge instructions (#17) * purge instructions * Apply suggestions from code review * Updated purge instructions Used the az commands, updated the pre-req language to remove the additional purge text. * Apply suggestions from code review * purge text changes Updated the purge instruction descriptions --------- Co-authored-by: Chad Kittel <[email protected]> * Minor cleanups, no functional changes * Quota related updates (#20) * Quota related updates * typo fixes * add transition note * Style changes * link updates * grammar * Add maxLength * Update boilerplate files * Images (#21) * Updated Images Updated PNG file to reflect naming changes * Updated Image Updated AI studio image * Update Readme image text Removed the TODO and clarified the alt image text * Updated Alt Text Added more descriptive text for the architecture drawing. * Update README.md --------- Co-authored-by: Chad Kittel <[email protected]> * Supported regions (#22) * region update Updates the regions to the tested working regions * Update README.md --------- Co-authored-by: Chad Kittel <[email protected]> * Update image to align with Microsoft Learn * update Learn article status --------- Co-authored-by: jchancellor-ms <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.