-
Notifications
You must be signed in to change notification settings - Fork 27
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: scale down while in use 🚨 #6898
✨Autoscaling: scale down while in use 🚨 #6898
Conversation
f4930bb
to
7004721
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6898 +/- ##
==========================================
+ Coverage 88.11% 88.15% +0.03%
==========================================
Files 1587 1580 -7
Lines 62156 61978 -178
Branches 2008 2008
==========================================
- Hits 54771 54636 -135
+ Misses 7050 7006 -44
- Partials 335 336 +1
Continue to review full report in Codecov by Sentry.
|
b5f913c
to
d6ad889
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.
Nice! Thanks a lot
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.
Didnt read the tests, rest looks very nice. one very minor comment
services/autoscaling/src/simcore_service_autoscaling/modules/auto_scaling_core.py
Outdated
Show resolved
Hide resolved
services/autoscaling/src/simcore_service_autoscaling/utils/utils_docker.py
Show resolved
Hide resolved
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.
👍
1a87a0b
to
3dcde20
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.
thx!
services/autoscaling/src/simcore_service_autoscaling/modules/auto_scaling_core.py
Outdated
Show resolved
Hide resolved
3dcde20
to
3a40819
Compare
Quality Gate passedIssues Measures |
What do these changes do?
Before this PR, the autoscaling would either scale UP or DOWN. Therefore if multiple tasks would be in the pipeline requiring one type of machine, and another set of tasks with a different type of machine came in the pipeline, the cluster scaling would deadlock as it would not be able to scale down the unused machines to make room for the machines required by the second set of tasks.
This PR aims to fix this:
🚨: This PR requires some manual testing to validate the approach works as expected
Bonus:
Related issue/s
How to test
Dev-ops checklist