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

feat(deadline): add ability to add spot event plugin managed policies to RenderQueue #141

Merged
merged 1 commit into from
Sep 30, 2020

Conversation

grbartel
Copy link
Contributor

Adds a helper function to the RenderQueue to add the requisite managed policies
for the SEP.

Testing

Tested by launching a renderfarm and configuring the SEP without setting any credentials in deadline to confirm the EC2 Instance role worked correctly.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Adds a helper function to the RenderQueue to add the requisite managed policies
for the SEP.
@jusiskin jusiskin changed the title feat(Deadline): Add abiility to Add SEP Manaaged Policies to RQ feat(Deadline): Add ability to Add SEP Managed Policies to RQ Sep 29, 2020
Copy link
Contributor

@ddneilson ddneilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple enough. Thanks for adding this Grant!

Copy link
Contributor

@jusiskin jusiskin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@jusiskin jusiskin changed the title feat(Deadline): Add ability to Add SEP Managed Policies to RQ feat(deadline): Add ability to Add SEP Managed Policies to RQ Sep 30, 2020
@jusiskin jusiskin changed the title feat(deadline): Add ability to Add SEP Managed Policies to RQ feat(deadline): add ability to add spot event plugin managed policies to RenderQueue Sep 30, 2020
@jusiskin jusiskin merged commit b2cf9e0 into aws:mainline Sep 30, 2020
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.

3 participants