Open
Conversation
- Demonstrates synchronous (polling) and callback integration patterns - Includes inline Python code in ECS task definitions - Complete SAM template with VPC, networking, and IAM roles - Comprehensive documentation and testing instructions
- Document permissive IAM and security group configurations - Explain why they exist (demo/learning purposes) - Provide production security recommendations - Recommend deployment in non-production environments
…ttern - Replace sfn-ecs-python-sam with lambda-ecs-python-sam - Implement synchronous pattern with polling using context.wait() - Implement callback pattern with DynamoDB state tracking - Use Python 3.13 with container image deployment - Support durable execution for workflows up to 1 year - Reduce costs with no charges during context.wait() periods
|
@4D54 instead of storing the callback token in dynamodb, the ecs task can call these to send the callback token to the lambda service directly? send_durable_execution_callback_failure |
Contributor
|
And please make sure to use the correct capitalization: It's Lambda durable functions not Lambda Durable Functions |
… APIs - callback_handler: Use context.create_callback() and callback.result() instead of DynamoDB-based polling - template.yaml: Remove CallbackTable, update ECSTaskRole with durable execution callback permissions, fix IAM circular dependency using Sub with explicit ARN construction - ECS task now calls SendDurableExecutionCallbackSuccess/Failure directly - README restructured to match serverless-patterns format
Author
|
Fixed! Thanks @rohanmeh @bfreiberg |
- Scope ecs:RunTask to specific task definitions instead of wildcard - Scope ecs:DescribeTasks to cluster tasks - Scope CheckpointDurableExecutions to specific function with durable-execution/* path - Fix directory name in README to lambda-ecs-durable-python-sam - Add expected test output for sync and callback patterns - Lowercase durable functions throughout docs - Add LinkedIn profile - Tested: both sync and callback patterns verified in CloudShell
Author
Contributor
|
Thanks @4D54 - adding this to the publishing queue. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.