-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
aws-ecs: support capacity provider #5471
Comments
What is the likelihood of this being added to cfn? |
Capacity Providers were just added to Cfn: 4ce27f4#diff-4a4e15a081904ee16b4d84e2f5cf5aee |
FYI: @SoManyHs |
This keeps on giving error |
See here about some issues with the current implementation: These are items that still planned to be implemented:
|
got the same issue in v1.53 :( |
@gandroz did you try putting asg name in the |
hmmm good catch @mb-dev, using How to link the capacity provider to the cluster ? |
Hi folks, I am working on the PR today and seems working fine in my integ test. See: https://twitter.com/pahudnet/status/1285440342045515777 Will create a PR draft shortly. |
We'll hold this issue because there are some feature gaps for CFN Capacity Provider to be narrowed. For example, the cyclic dependency that is already discussed here. Even if we try to break the cyclic dependency by using the hard code cluster name, we'll still have problems when deleting resources. Hence, we decided to hold this issue and wait for CFN update on Capacity Provider before we support it in CDK. |
@iamhopaul123 I believe this may have been unblocked recently? I'm not sure if it only applies to Fargate and whether it can work with EC2 as well. |
Hi @andreialecu thank you so much for bringing this great news to us! We'll take a look at it to see if it could fit in fixing this issue. |
Hello @andreialecu, looks like number 4 and 5 of aws/containers-roadmap#631 (comment) are still waiting to be fixed. Unfortunately though CFN support could simplify the CDK support for quite a lot, without fixing the circular dependency problem there is not much we could do. We'll keep waiting until there are more updates on this. |
Hello everyone. Is there any news on this issue? |
Is this coming soon? |
Hello, we don't have any update for now. We'll update this issue we there's any news. |
We are interest in this too, mainly due possibilities to use launch templates. I think I’ve got idea how to solve it and I did preliminary tests. We could create custom cloud formation resource “CapacityProviderAttachment”, which will have two inputs cluster & capacity providers - this is simple lambda. Custom resource could be part of “bootstrap” or some kind of inner stack (I prefer later as CF templates are commonly used). The resource would call put-capacity-providers on creation and remove from cluster on deletion. I would have to validate deletion only, to have final results. |
Supporting EC2 Capacity Providers may now be possible, now that the AWS::ECS::ClusterCapacityProviderAssociations resource is available. |
I missed this one. So this ticket and bunch of other leading to launch templates can be shortly solved.`` |
Add support for EC2 Capacity Providers. I tried to make the UX reasonably decent without disrupting existing stacks. There are some new types being created here, including `ICapacityProvider` (interface type) and its concrete type `EC2CapacityProvider` (which is a class users can instantiate). There are also a couple of singleton classes for `FargateCapacityProvider` and `FargateSpotCapacityProvider`. Closes #5471 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
Add support for EC2 Capacity Providers. I tried to make the UX reasonably decent without disrupting existing stacks. There are some new types being created here, including `ICapacityProvider` (interface type) and its concrete type `EC2CapacityProvider` (which is a class users can instantiate). There are also a couple of singleton classes for `FargateCapacityProvider` and `FargateSpotCapacityProvider`. Closes aws#5471 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
https://aws.amazon.com/tw/about-aws/whats-new/2019/12/amazon-ecs-capacity-providers-now-available/
Use Case
Proposed Solution
Other
dependency aws/containers-roadmap#631
This is a 🚀 Feature Request
The text was updated successfully, but these errors were encountered: