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

♻️Autoscaling: set default termination time of EC2 instance to 1 minute #3970

Conversation

sanderegg
Copy link
Member

@sanderegg sanderegg commented Mar 14, 2023

What do these changes do?

fixes ITISFoundation/osparc-issues#657

as explained there:
https://aws.amazon.com/ec2/pricing/on-demand/

The price of EC2 instances is billed by the second. therefore the default for terminating a node is changed from 55 minutes to 1 minute.

Related issue/s

How to test

Checklist

@sanderegg sanderegg added changelog:♻️refactor a:autoscaling autoscaling service in simcore's stack labels Mar 14, 2023
@sanderegg sanderegg added this to the Mithril milestone Mar 14, 2023
@sanderegg sanderegg self-assigned this Mar 14, 2023
@codeclimate
Copy link

codeclimate bot commented Mar 14, 2023

Code Climate has analyzed commit db563e5 and detected 0 issues on this pull request.

View more on Code Climate.

@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Merging #3970 (db563e5) into master (d7f1ce5) will decrease coverage by 14.9%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #3970      +/-   ##
=========================================
- Coverage    83.0%   68.1%   -14.9%     
=========================================
  Files         943     413     -530     
  Lines       40665   21128   -19537     
  Branches      848     137     -711     
=========================================
- Hits        33764   14407   -19357     
+ Misses       6691    6672      -19     
+ Partials      210      49     -161     
Flag Coverage Δ
integrationtests 66.7% <ø> (+2.6%) ⬆️
unittests 98.2% <ø> (+17.2%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...g/src/simcore_service_autoscaling/core/settings.py 100.0% <ø> (ø)

... and 756 files with indirect coverage changes

@sonarcloud
Copy link

sonarcloud bot commented Mar 14, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sanderegg sanderegg merged commit b62985f into ITISFoundation:master Mar 14, 2023
@sanderegg sanderegg deleted the autoscaling/terminate_ec2_instances_after_1_minute branch March 14, 2023 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:autoscaling autoscaling service in simcore's stack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Autoscaling - Dynamic Services
5 participants