-
Notifications
You must be signed in to change notification settings - Fork 244
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
Logging improvements #5749
base: development/8.8
Are you sure you want to change the base?
Logging improvements #5749
Conversation
- From info to debug when applicable - From info to warn or error Issue: CLDSRV-613
This does not prevent any race condition but at least leaves a log when it happens. Issue: CLDSRV-613
Hello williamlardier,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files
... and 1 file with indirect coverage changes @@ Coverage Diff @@
## development/8.8 #5749 +/- ##
===================================================
+ Coverage 64.46% 65.30% +0.83%
===================================================
Files 185 185
Lines 11638 11643 +5
===================================================
+ Hits 7502 7603 +101
+ Misses 4136 4040 -96
Flags with carried forward coverage won't be shown. Click here to find out more. |
Issue: CLDSRV-613
773d4a9
to
acb99f2
Compare
ba341b5
to
3b3a567
Compare
3b3a567
to
18cac34
Compare
Improve logging to use better levels, or add logs that were missing in
routeBackbeat
.The delete operation also will log a warning when the expiration tries to delete a master version (due to any race condition).
Also bump Arsenal from scality/Arsenal#2300