Skip to content

New serverless pattern: s3-lambda-dynamodb-terraform #2513

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

Merged
merged 14 commits into from
Apr 23, 2025

Conversation

sarika-subram
Copy link
Contributor

Issue #, if available:

Description of changes: New serverless pattern - s3-lambda-dynamodb-terraform

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@biswanathmukherjee
Copy link
Contributor

Hi @sarika-subram ,
Thanks for your contribution. I see an existing similar pattern: https://serverlessland.com/patterns/s3-lambda-dynamodb-cdk
Is this the same pattern using Terraform? If yes, please use same name.

Also you need to have example-pattern.json file like this sample below.

https://github.com/aws-samples/serverless-patterns/blob/main/_pattern-model/example-pattern.json

@sarika-subram
Copy link
Contributor Author

Thanks @biswanathmukherjee ,
Have added the pattern.json file.

It is the same pattern that uses Terraform and inputs json instead of csv. Could you please direct me to where the name should be changed?

@sarika-subram
Copy link
Contributor Author

Made all requested changes. Please review the updates and let me know if any further changes are needed.

@biswanathmukherjee
Copy link
Contributor

Approved from my side. One of the DAs will merge it soon. Thanks for your contribution on serverlessland.

@julianwood julianwood merged commit 93e7b3e into aws-samples:main Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants