Add support of capacity provider strategy for ECSOperator #15848
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.
This PR adds support of support of capacity provider strategy for ECSOperator. The logic is based on ECS.Client.run_task documentation.
Closes #8381
Note:
There is a logic related to the
platform_version
that I find tricky:As for me, the operator should pass to the
run_task
every parameter that user provided. And that should be responsibility of the user to think about which parameters and under which conditions are needed. But to be consistent with the logic, I also provideplatform_version
ifcapacity_provider_strategy
is specified.capacity_provider_strategy
can be used only with Fargate, so the platform version should be passed as well. But as we can see, picking the parameters introduces an unnecessary complexity.