-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
switch ecs to use fargate_spot #1072
base: master
Are you sure you want to change the base?
Conversation
Since I cannot add you as a reviewer, I'm pinging here directly 😄 @drAlberT what do you think? |
74545ab
to
3990d93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
uh? Why?
As usual for any use of SPOT resources: if all the jobs that are running are interruptions' tolerant then it's of course the right choice, if having a job brutally interrupted could be an issue then SPOT instances are not a choice (and the workloads should be adjusted to try being tolerant, making it possible). ;) |
3990d93
to
3cc3dd4
Compare
I'll try deploying this at the next application rollout. |
504a672
to
263c5cc
Compare
@fabiocicerchia merge or close? |
@JellyBellyDev I'll merge after testing it out while deploying the next release |
current pricing for FARGATE could be reduced by 70% with FARGATE_SPOT.