misc: add a debug level mode for cron logs (#88912) #1349

Merged
fpeters merged 1 commits from wip/88912-cron-debug-memory-logs into main 2024-04-03 10:55:48 +02:00
Owner
No description provided.
fpeters added 1 commit 2024-04-01 18:12:01 +02:00
gitea/wcs/pipeline/head This commit looks good Details
454f690594
misc: add a debug level mode for cron logs (#88912)
fpeters force-pushed wip/88912-cron-debug-memory-logs from 454f690594 to 3354e27bda 2024-04-02 08:41:10 +02:00 Compare
fpeters force-pushed wip/88912-cron-debug-memory-logs from 3354e27bda to 4f627885f7 2024-04-02 10:24:16 +02:00 Compare
fpeters changed title from WIP: misc: add a debug level mode for cron logs (#88912) to misc: add a debug level mode for cron logs (#88912) 2024-04-02 11:32:24 +02:00
tnoel approved these changes 2024-04-02 15:23:30 +02:00
tnoel left a comment
Owner

Mini commentaire sur le message loggué par log_debug... mais je valide quand même parce que c'est mini.

Mini commentaire sur le message loggué par log_debug... mais je valide quand même parce que c'est mini.
@ -79,0 +81,4 @@
if get_publisher().get_site_option('cron-log-level') != 'debug':
return
memory = psutil.Process().memory_info().rss / (1024 * 1024)
self.log(f'(mem: {memory:.1f}M) {message}', in_tenant=in_tenant)
Owner

Peut-être, pour se rappeler que c'est du niveau debug, ajouter un [debug] au niveau du message ?

Peut-être, pour se rappeler que c'est du niveau debug, ajouter un [debug] au niveau du message ?
Author
Owner

Je me dis que juste le (mem) fait cette indication. (et ça me permettait de tout juste tenir sur la largeur de mon terminal…).

Je me dis que juste le (mem) fait cette indication. (et ça me permettait de tout juste tenir sur la largeur de mon terminal…).
fpeters merged commit 4e269e532f into main 2024-04-03 10:55:48 +02:00
fpeters deleted branch wip/88912-cron-debug-memory-logs 2024-04-03 10:55:48 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1349
No description provided.