SQL Lambda Tenant Isolation#2968
Conversation
|
@marcojahn @julianwood Do you have update on this PR? |
marcojahn
left a comment
There was a problem hiding this comment.
Hello @devops-arch-cloud, I've added a review comment, pls review.
devops-arch-cloud
left a comment
There was a problem hiding this comment.
Updated read me file
devops-arch-cloud
left a comment
There was a problem hiding this comment.
Updated readme file
marcojahn
left a comment
There was a problem hiding this comment.
Hello @devops-arch-cloud, i've added a few comments, pls review
Co-authored-by: Marco <marco.jahn@gmail.com>
Co-authored-by: Marco <marco.jahn@gmail.com>
Co-authored-by: Marco <marco.jahn@gmail.com>
Co-authored-by: Marco <marco.jahn@gmail.com>
marcojahn
left a comment
There was a problem hiding this comment.
Hello @devops-arch-cloud,
i've added a few comments, please review?
Co-authored-by: Marco <marco.jahn@gmail.com>
devops-arch-cloud
left a comment
There was a problem hiding this comment.
Updated the requested changes
|
@marcojahn let me know if any other changes are required |
marcojahn
left a comment
There was a problem hiding this comment.
Hello @devops-arch-cloud, one small fix, you are using AWS SAM for provisioning, so you can use it for stack deletion as well. Besides this, this pattern is good to go
added sqs-lambda-tenant-isolation-sam-py.json
Co-authored-by: Marco <marco.jahn@gmail.com>
added small code fix
marcojahn
left a comment
There was a problem hiding this comment.
Hi @devops-arch-cloud, thanks for your contribution. Your PR will be merged soon
Issue #, if available:
Description of changes:
Multi-tenant application demonstrating AWS Lambda's tenant isolation feature.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.