Skip to content
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

feat: add support for running on Fargate Spot #2464

Conversation

hassy
Copy link
Member

@hassy hassy commented Feb 2, 2024

No description provided.

@bernardobridge
Copy link
Contributor

bernardobridge commented Feb 2, 2024

Does this work for existing clusters (like the default one)? I thought we would have to leverage checkTargetCluster which is run before anyway, and then if no capacity provider is in the default cluster, we could add it there.

If/When it does work for existing cluster, it might be worth just altering one of the tests to run on spot, to give us free testing of this feature?

@@ -889,7 +890,7 @@ async function createArtilleryCluster(context) {
await ecs
.createCluster({
clusterName: ARTILLERY_CLUSTER_NAME,
capacityProviders: ['FARGATE']
capacityProviders: ['FARGATE_SPOT']
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Setting this as the default provider allows scheduling on-demand Fargate tasks but not vice versa

@hassy
Copy link
Member Author

hassy commented Feb 2, 2024

@bernardobridge existing clusters have to be recreated, will be in the docs

Cluster in us-east-2 in the test AWS account created with default FARGATE_SPOT
capacity provided.
@hassy hassy merged commit 35f0e1c into main Feb 3, 2024
18 checks passed
@hassy hassy deleted the hassy-art-1611-add-support-for-running-on-fargate-spot-capacity-providers branch February 3, 2024 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants