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

Modified policy to support assume role for MLfoundry as well #5

Merged
merged 5 commits into from
Oct 17, 2023
Merged

Modified policy to support assume role for MLfoundry as well #5

merged 5 commits into from
Oct 17, 2023

Conversation

dunefro
Copy link
Contributor

@dunefro dunefro commented Oct 17, 2023

  1. Add support for assume role in ML foundry role
  2. Migrate policy to be truefoundry specific.

@dunefro dunefro self-assigned this Oct 17, 2023
shubhamrai1993
shubhamrai1993 previously approved these changes Oct 17, 2023
View reviewed changes
locals.tf Outdated Show resolved Hide resolved
@dunefro dunefro marked this pull request as ready for review October 17, 2023 11:39
@dunefro dunefro merged commit f121577 into main Oct 17, 2023
@dunefro dunefro deleted the mlfoundry-policy branch October 17, 2023 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shubhamrai1993 shubhamrai1993 shubhamrai1993 approved these changes

Assignees

@dunefro dunefro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dunefro @shubhamrai1993