Skip to content

Commit 8abf72b

Browse files
Merge pull request #45843 from nextcloud/backport/45530/stable29
[stable29] fix(cron): Log excess memory usage at debug and info levels
2 parents 79721b4 + 92fce66 commit 8abf72b

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
@@ -197,10 +197,10 @@
197197
);
198198
}
199199

200-
if ($memoryAfter - $memoryBefore > 10_000_000) {
201-
$logger->warning('Used memory grew by more than 10 MB when executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryAfter). ' (before: ' . Util::humanFileSize($memoryBefore) . ')', ['app' => 'cron']);
200+
if ($memoryAfter - $memoryBefore > 50_000_000) {
201+
$logger->warning('Used memory grew by more than 50 MB when executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryAfter). ' (before: ' . Util::humanFileSize($memoryBefore) . ')', ['app' => 'cron']);
202202
}
203-
if ($memoryPeakAfter > 300_000_000) {
203+
if ($memoryPeakAfter > 300_000_000 && $memoryPeakBefore <= 300_000_000) {
204204
$logger->warning('Cron job used more than 300 MB of ram after executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryPeakAfter) . ' (before: ' . Util::humanFileSize($memoryPeakBefore) . ')', ['app' => 'cron']);
205205
}
206206

0 commit comments

Comments
 (0)