Skip to content

fix(cron): Log excess memory usage at debug and info levels - #45530

Merged
joshtrichards merged 2 commits into
masterfrom
jtr/fix-cron-memory-usage-levels
Jun 12, 2024
Merged

fix(cron): Log excess memory usage at debug and info levels#45530
joshtrichards merged 2 commits into
masterfrom
jtr/fix-cron-memory-usage-levels

Conversation

@joshtrichards

@joshtrichardsjoshtrichards commented May 27, 2024

Copy link
Copy Markdown
Member

Summary

Admins/operators don't need to see this in day-to-day scenarios >= the default log level. Moved the 10MiB (common) to DEBUG level and the 300 MiB to INFO level (rarer but still common enough).

TODO

Checklist

@joshtrichards

Copy link
Copy Markdown
MemberAuthor

/backport to stable29

Comment threadcron.php Outdated
Comment threadcron.php Outdated
Comment threadcron.php Outdated
@github-actions

This comment was marked as off-topic.

Signed-off-by: Josh <josh.t.richards@gmail.com>
@nickvergessen
nickvergessenforce-pushed the jtr/fix-cron-memory-usage-levels branch from 6cae530 to 8ee070cCompareJune 11, 2024 12:21
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichardsjoshtrichards added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 12, 2024
@joshtrichards
joshtrichards merged commit a11262b into masterJun 12, 2024
@joshtrichards
joshtrichards deleted the jtr/fix-cron-memory-usage-levels branch June 12, 2024 22:26
@blizzzblizzz mentioned this pull request Jul 24, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: loggingfeedback-requested

Projects

None yet

6 participants

@joshtrichards@nickvergessen@ChristophWurst@szaimen@vvp1983@come-nc