Skip to content

Commit 6cae530

Browse files
joshtrichardsnickvergessen
authored andcommitted
fix(cron): Log excess memory usage at debug and info levels
Signed-off-by: Josh <josh.t.richards@gmail.com>
1 parent bab9125 commit 6cae530

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

cron.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -163,10 +163,10 @@
163163
$memoryAfter = memory_get_usage();
164164
$memoryPeakAfter = memory_get_peak_usage();
165165

166-
if ($memoryAfter - $memoryBefore > 10_000_000) {
167-
$logger->warning('Used memory grew by more than 10 MB when executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryAfter). ' (before: ' . Util::humanFileSize($memoryBefore) . ')', ['app' => 'cron']);
166+
if ($memoryAfter - $memoryBefore > 50_000_000) {
167+
$logger->warning('Used memory grew by more than 50 MB when executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryAfter). ' (before: ' . Util::humanFileSize($memoryBefore) . ')', ['app' => 'cron']);
168168
}
169-
if ($memoryPeakAfter > 300_000_000) {
169+
if ($memoryPeakAfter > 300_000_000 && $memoryPeakBefore <= 300_000_000) {
170170
$logger->warning('Cron job used more than 300 MB of ram after executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryPeakAfter) . ' (before: ' . Util::humanFileSize($memoryPeakBefore) . ')', ['app' => 'cron']);
171171
}
172172

0 commit comments

Comments
 (0)