diff options
author | Josh <josh.t.richards@gmail.com> | 2024-06-12 18:26:33 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-06-12 18:26:33 -0400 |
commit | a11262b212103327584dab89e9a36a932918bff9 (patch) | |
tree | 7cbdc96f6e9f76f815ab0caa9153a0cd49a3ad34 | |
parent | b3e4270135d5de64b2b4d43b3d08f00c91b7beab (diff) | |
parent | ecef3922317cc28e6d34c609e266b68159c2146e (diff) | |
download | nextcloud-server-a11262b212103327584dab89e9a36a932918bff9.tar.gz nextcloud-server-a11262b212103327584dab89e9a36a932918bff9.zip |
Merge pull request #45530 from nextcloud/jtr/fix-cron-memory-usage-levels
fix(cron): Log excess memory usage at debug and info levels
-rw-r--r-- | cron.php | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -182,10 +182,10 @@ Options: ); } - if ($memoryAfter - $memoryBefore > 10_000_000) { - $logger->warning('Used memory grew by more than 10 MB when executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryAfter). ' (before: ' . Util::humanFileSize($memoryBefore) . ')', ['app' => 'cron']); + if ($memoryAfter - $memoryBefore > 50_000_000) { + $logger->warning('Used memory grew by more than 50 MB when executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryAfter). ' (before: ' . Util::humanFileSize($memoryBefore) . ')', ['app' => 'cron']); } - if ($memoryPeakAfter > 300_000_000) { + if ($memoryPeakAfter > 300_000_000 && $memoryPeakBefore <= 300_000_000) { $logger->warning('Cron job used more than 300 MB of ram after executing job ' . $jobDetails . ': ' . Util::humanFileSize($memoryPeakAfter) . ' (before: ' . Util::humanFileSize($memoryPeakBefore) . ')', ['app' => 'cron']); } |