Skip to content

Commit

Permalink
feat(ecs-patterns): healthCheck property for `ApplicationLoadBalanc…
Browse files Browse the repository at this point in the history
…edFargateService` (#28797)

Closes #28796

Like with other `aws-ecs-pattern` high level constructs, some properties aren't exposed. For `aws_ecs_patterns`.`ApplicationLoadBalancedFargateService` this is `healthCheck` property of the container. 

There are already similar PRs #18219 for other constructs in the `aws_ecs_patterns` lib

*Note*:

I didn't add any test cases since `ApplicationLoadBalancedFargateService` does not seem to have them like the others do. I can maybe come back with another PR to add the test cases.

I didn't update the docs since it's a minor addition. And there is already an example for the `QueueProcessingFargateService` that uses the `healthCheck`, it might be redundant to add the same code example with different constructor name. 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
Nikola-Milovic authored and TheRealAmazonKendra committed Feb 9, 2024
1 parent 6e3f2de commit 7730f07
Show file tree
Hide file tree
Showing 12 changed files with 2,646 additions and 1 deletion.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 7730f07

Please sign in to comment.