aws-stepfunctions-tasks: Allow using a specific revision in EcsRunTask

[AdamJHall]

Describe the feature

Currently when defining a EcsRunTask you pass through a TaskDefinition however the resulting stepfunction only references the task definitions family and the step function will use the latest "Active" definition.

However I would like to specify a revision number so that I can reference older task definition versions. This does seem to be possible with the underlying cloudformation but not via CDK.

Use Case

I have one stack which defines the task definition and another stack that accepts that stack as a prop and deploys a step function using that task definition.

There will be multiple instances of the stepfunction stack and I want that stepfunctions created by those stacks to refer to that task definition that it was referencing at that point in time.

Proposed Solution

Something like the below would be ideal, where revisionNumber is optional to not break current behaviour.

const runTask = new EcsRunTask(this, "RunTask", {
    cluster,
    taskDefinition,
    revisionNumber: taskDefinition.revisionNumber,
    launchTarget: new EcsFargateLaunchTarget(),
});

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

CDK version used

2.43.1

Environment details (OS name and version, etc.)

Ubuntu 20.04

[peterwoodworth]

According to the API ref, we should specify the task definition with family:revision. Currently, we only specify family. It should be easy to append the revision here if specified, thanks for the request!

I am marking this issue as p2, which means that we are unable to work on this immediately.

We use +1s to help prioritize our work, and are happy to revaluate this issue based on community feedback. You can reach out to the cdk.dev community on Slack to solicit support for reprioritization.

Check out our contributing guide if you're interested in contributing yourself - there's a low chance the team will be able to address this soon but we can try to review a PR 🙂

[zorrofox]

I have a question about this requirement, because if we use different Revision Number for TaskDefinition there will be different TaskDefinition instance and how we can modified the EcsRunTask property taskDefinition?

[westhouseK]

@peterwoodworth
May I work on it? I want to try it!
I'm going to fix TaskDefinition, is it ok?

[peterwoodworth]

Yes, go ahead @westhouseK! Be sure to add unit and integration testing, and to follow the contributing guide I linked above!

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.