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(bootstrap): Add ECR resource policy for emr-serverless containers #112

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

graydenshand
Copy link

@graydenshand graydenshand commented Feb 24, 2025

This PR modifies the default bootstrap template to support EMR Serverless containers.

Without this policy, containers created using the DockerImageAsset construct cannot be used with EMR Serverless applications.

There is precedence for this pattern to support Lambda function containers, so I don't think this should be too controversial of a change.

Relevant documentation on using custom images in EMR Serverless:

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

@graydenshand graydenshand changed the title Bootstrap template | Add ECR resource policy for emr-serverless containers feat (bootstrap): Add ECR resource policy for emr-serverless containers Feb 24, 2025
@graydenshand graydenshand changed the title feat (bootstrap): Add ECR resource policy for emr-serverless containers feat(bootstrap): Add ECR resource policy for emr-serverless containers Feb 24, 2025
@graydenshand
Copy link
Author

@mrgrain @kaizencc @rix0rrr Anyone able to take a look at this?

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.

1 participant