-
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
🎨 reduce noisy logs in director-v2
#5300
🎨 reduce noisy logs in director-v2
#5300
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #5300 +/- ##
=========================================
- Coverage 83.0% 65.1% -18.0%
=========================================
Files 1239 542 -697
Lines 50972 27146 -23826
Branches 1127 202 -925
=========================================
- Hits 42356 17696 -24660
- Misses 8376 9400 +1024
+ Partials 240 50 -190
Flags with carried forward coverage won't be shown. Click here to find out more.
|
...-v2/src/simcore_service_director_v2/modules/dynamic_sidecar/scheduler/_core/_events_utils.py
Outdated
Show resolved
Hide resolved
...-v2/src/simcore_service_director_v2/modules/dynamic_sidecar/scheduler/_core/_events_utils.py
Outdated
Show resolved
Hide resolved
…s-noise-reduction
…/osparc-simcore-forked into pr-osparc-dv2-logs-noise-reduction
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
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
What do these changes do?
When the method
attempt_pod_removal_and_data_saving
is called, it is not guaranteed that the dynamic-sidecar is present. Theservice_free_reserved_disk_space
was not taking this into account. It is now in line withservice_remove_containers
.Removes these not so useful tracebacks from director-v2
Click to see error traceback sample
Related issue/s
How to test
Dev Checklist
DevOps Checklist