Re: [pbs-devel] [PATCH proxmox{, -backup} v5 0/4] proxmox-log introduction

[Lukas Wagner <l.wagner@proxmox.com>]

On  2024-06-13 15:56, Gabriel Goller wrote:
> Removed the task_log! (and friends task_warn!, task_debug!, etc.) macro
> and introduced the `tracing` crate. We now initiate the tracing crate
> using a single layer, which is logging to the syslog and the tasklog.
> It uses the `syslog` crate and the original `FileLogger`.
> 
> To write to the task logs from the worker threads and tasks, we now
> have a task_local logger (and warning counter), which
> will get instantiated when a task/thread is created. This means that
> when we call `info!` or any other `tracing` log macros, it will get 
> the file_logger from TLS and write to the file.
> 
> 
> 
> Summary over all repositories:
>   46 files changed, 828 insertions(+), 1046 deletions(-)
> 

Great work!

Applied this on latest master and did a quick sanity check to ensure
that (task) logging still works. 

I left some comments with a few minor things to improve. With those fixed,
consider this:

Tested-by: Lukas Wagner <l.wagner@proxmox.com>
Reviewed-by: Lukas Wagner <l.wagner@proxmox.com>

-- 
- Lukas


_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel