adding lambda-durable-functions-nodejs-calling-ecs#2959
Open
dsiz wants to merge 5 commits intoaws-samples:mainfrom
Open
adding lambda-durable-functions-nodejs-calling-ecs#2959dsiz wants to merge 5 commits intoaws-samples:mainfrom
dsiz wants to merge 5 commits intoaws-samples:mainfrom
Conversation
bfreiberg
requested changes
Feb 20, 2026
| "description": "Orchestrate long-running ECS Fargate tasks using Lambda durable functions with callback pattern", | ||
| "language": "Node.js", | ||
| "level": "200", | ||
| "framework": "SAM", |
Contributor
There was a problem hiding this comment.
Suggested change
| "framework": "SAM", | |
| "framework": "AWS SAM", |
| "headline": "How it works", | ||
| "text": [ | ||
| "This pattern demonstrates how to use AWS Lambda durable functions to orchestrate Amazon ECS Fargate tasks that can run for up to 24 hours.", | ||
| "The Lambda function creates a callback ID, starts an ECS task with that ID, and suspends execution without incurring compute charges during the wait period.", |
Contributor
There was a problem hiding this comment.
Suggested change
| "The Lambda function creates a callback ID, starts an ECS task with that ID, and suspends execution without incurring compute charges during the wait period.", | |
| "The Lambda function creates a callback ID, starts an ECS task with that ID, and then automatically suspends execution without incurring compute charges during the wait period.", |
| }, | ||
| "cleanup": { | ||
| "text": [ | ||
| "sam delete" |
Contributor
There was a problem hiding this comment.
Suggested change
| "sam delete" | |
| "<code>sam delete</code>" |
| { | ||
| "name": "Surya Sai D", | ||
| "image": "", | ||
| "bio": "Surya works as a Technical Account Manager in AWS. He is an expert in Serverless frameworks and Event Driven Architectures. Surya is also passionate on technical writing and has contributed to AWS blogs and other Open Source Content.", |
Contributor
There was a problem hiding this comment.
Suggested change
| "bio": "Surya works as a Technical Account Manager in AWS. He is an expert in Serverless frameworks and Event Driven Architectures. Surya is also passionate on technical writing and has contributed to AWS blogs and other Open Source Content.", | |
| "bio": "Surya works as a Technical Account Manager at AWS. He is an expert in Serverless frameworks and Event Driven Architectures. Surya is also passionate on technical writing and has contributed to AWS blogs and other Open Source Content.", |
| response.json | ||
| ``` | ||
|
|
||
| **Note:** The `:prod` alias is required for durable functions. |
Contributor
There was a problem hiding this comment.
The alias is not required. Any qualified ARN suffices (https://docs.aws.amazon.com/lambda/latest/dg/durable-invoking.html#durable-invoking-qualified-arns)
| Action: | ||
| - ecs:RunTask | ||
| - ecs:DescribeTasks | ||
| Resource: '*' |
Contributor
There was a problem hiding this comment.
restrict to same account and current region
| - logs:CreateLogGroup | ||
| - logs:CreateLogStream | ||
| - logs:PutLogEvents | ||
| Resource: '*' |
Contributor
There was a problem hiding this comment.
restrict to same account and current region
| ExecutionTimeout: 86400 | ||
| RetentionPeriodInDays: 7 | ||
|
|
||
| # Lambda Durable Function - Polling Pattern |
lambda-durable-functions-nodejs-calling-ecs/src/callback-handler.js
Outdated
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
This file does not seem related to this pattern
dsiz
force-pushed
the
dsiz-feature-lambda-durable-functions-nodejs-calling-ecs
branch
from
2fbfbf4 to
dc9dd7b
Compare
Contributor
Author
|
@bfreiberg please check again, I applied the suggested changes and also removed polling pattern to stick with callback pattern only. |
bfreiberg
requested changes
Feb 22, 2026
Contributor
bfreiberg
left a comment
bfreiberg
left a comment
There was a problem hiding this comment.
Looks good, one minor change and one open comment from the previous round.
| @@ -0,0 +1,68 @@ | |||
| { | |||
| "title": "AWS Lambda durable functions in NodeJS calling ECS", | |||
Contributor
There was a problem hiding this comment.
Suggested change
| "title": "AWS Lambda durable functions in NodeJS calling ECS", | |
| "title": "AWS Lambda durable functions in NodeJS calling Amazon ECS", |
bfreiberg
reviewed
Feb 22, 2026
Comment on lines
+28
to
+29
| console.log('Callback ID created:', callbackId); | ||
| console.log('Starting ECS task with callback ID...'); |
Contributor
There was a problem hiding this comment.
Please change all console.log to context.logger (see https://github.com/aws/aws-durable-execution-sdk-js/tree/main)
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
3 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:
adding lambda-durable-functions-nodejs-calling-ecs
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.