aws-ecs: Should QueueProcessingFargateService support healthCheckGracePeriod?

[TrevorBurnham]

Describe the feature

QueueProcessingFargateService supports the circuitBreaker prop. However, unlike other constructs that support circuitBreaker (such as FargateService), it doesn't support the healthCheckGracePeriod prop. That means that, as far as I can tell, there's no way to configure the grace period for the circuit breaker in a QueueProcessingFargateService.

Use Case

I'm seeing premature deployment rollbacks in a QueueProcessingFargateService with the deployment circuit breaker enabled. I'd like to extend the grace period to prevent these premature rollbacks.

Proposed Solution

The healthCheckGracePeriod option on QueueProcessingFargateService should match the option on FargateService.

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.190.0

Environment details (OS name and version, etc.)

N/A

[pahud]

Hi @TrevorBurnham,

Thank you for your suggestion and detailed report!

You're correct: while FargateService supports healthCheckGracePeriod, QueueProcessingFargateService currently does not expose or forward this property to its inner service, even when the circuitBreaker logic would benefit from it.

Proposed solution:

• Add a healthCheckGracePeriod property to the QueueProcessingFargateServiceProps.
• Pass this property in the constructor when creating the FargateService inside QueueProcessingFargateService.

This would allow for a consistent user experience and help avoid premature deployment rollbacks due to short health check grace periods.

We'd welcome a Pull Request for this enhancement! If you're interested, let us know and we can help guide you through the changes. Otherwise, this is a reasonable feature request that aligns with how other ECS constructs currently work.

Thanks again for helping us improve the CDK!

(For reference, similar property handling can be seen in BaseService.)

[TrevorBurnham]

I've sent over a CR: #34231 However, I'm having some issues getting the tests to run. See this comment for details.