[stable29] fix(cron): Log excess memory usage at debug and info levels

[backportbot]

Backport of #45530

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

• Review and resolve any conflicts
• Amend HEAD commit to remove the line stating to skip CI

---

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

[joshtrichards]

/update-3rdparty

[szaimen]

🐘

[nickvergessen]

The log you posted will not happen in the future anymore, because && $memoryPeakBefore <= 300_000_000 is not true as the before: 554.9 MB is not smaller than 300 MB

[AndyXheli]

Hi still seeing this on NC 29.0.4

{"reqId":"kImcJ5WNoR8EST80crIE","level":2,"time":"2024-07-28T21:17:03-05:00","remoteAddr":"","user":"--","app":"cron","method":"","url":"--","message":"Cron job used more than 300 MB of ram after executing job OCA\\Recognize\\BackgroundJobs\\ClusterFacesJob (id: 226426, arguments: {\"userId\":\"admin\"}): 652.1 MB (before: 99.5 MB)","userAgent":"--","version":"29.0.4.1","data":{"app":"cron"},"id":"66a7bdec119fb"}
{"reqId":"IXdi9NgDcaj3o5rTidfQ","level":2,"time":"2024-07-28T21:01:47-05:00","remoteAddr":"","user":"--","app":"cron","method":"","url":"--","message":"Cron job used more than 300 MB of ram after executing job OCA\\Recognize\\BackgroundJobs\\ClusterFacesJob (id: 226389, arguments: {\"userId\":\"admin\"}): 625.5 MB (before: 10.6 MB)","userAgent":"--","version":"29.0.4.1","data":{"app":"cron"},"id":"66a7bdec11a7f"}

[joshtrichards]

Yes, the logging is legitimate. It's to catch large memory consumers. Report it to the recognize repo please, since that's the job that is using the memory in your case.

[nickvergessen]

In your sample the memory usage increased by 615MB and 550MB. Exactly the cases we want to log. 🤷