-
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
✨🎨 Preferences: Inactivity and Concurrency #5024
✨🎨 Preferences: Inactivity and Concurrency #5024
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #5024 +/- ##
=========================================
- Coverage 84.8% 81.2% -3.6%
=========================================
Files 1263 526 -737
Lines 52019 26670 -25349
Branches 1124 198 -926
=========================================
- Hits 44113 21675 -22438
+ Misses 7673 4945 -2728
+ Partials 233 50 -183
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Very nice! Some questions/comments:
Can the "inactivity shutdown" be set to None (so it never happens)? If this is possible,this we should also warn the users that this going to consume credits.
It is not very clear what the concurrent job preference does. We might add a short explanation.
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.
👍
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.
👍
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!
for @elisabettai the max concurrent job should allow:
- say you have a pipeline with 10 parallel computational services
- if max concurrent jobs is set to 4, then only 4 will run in parallel (each service consumes 1 machine),
- that also means that if you run this via publicAPI, at most 4 jobs will run in parallel,
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 👍
…osparc-simcore into inactivity-concurrency-settings
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Code Climate has analyzed commit f43cf5f and detected 0 issues on this pull request. View more on Code Climate. |
What do these changes do?
Related issue/s
How to test
Check the preferences dialog for the mentioned properties and check that they change in the DB.
Please review the wordings.
Dev Checklist
DevOps Checklist