Add option to specify lambda via image_uri #15
Merged
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.
Are there any dependencies (software or human) that need to be addressed before this PR is merged?
What ticket(s) or other PRs does this relate to?
What was the problem or feature?
Want to be able to create a Lambda based on an image in ECR.
What was the solution?
Add the
image_uri
option to the variables and to the creation of the Lambda resource.Additionally add verifications and conditionals to ensure that we don't specify conflicting S3 and ECR configurations.(this commit, we can add these once we get to Terraform > 1.2.0)Where does this work fall on the Good - Fast spectrum?
Any additional work needed as a result of merging this PR (deploy steps, other PRs, etc.)?