On  2024-12-05 16:10, Gabriel Goller wrote:
> Add special logger for perlmod. This one will print everything to
> stderr (which will end up in the tasklog) and the errors to journald.
> This also includes a custom formatter so that the stderr output will
> look like: "INFO: sample message".
> 
> Reported-by: Maximiliano Sandoval <m.sando...@proxmox.com>
> Reported-by: Lukas Wagner <l.wag...@proxmox.com>
> Signed-off-by: Gabriel Goller <g.gol...@proxmox.com>
> ---
> 
> I didn't include Lukas's T-b as I changed quite a lot on this 
> patch.
> 

Gave v3 another quick spin, seems to work as advertised. Code also
looks good to me, though I have to say that I don't know much about
the tracing ecosystem.

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

Maybe another set of eyes would be good, as I'm not sure if I'm overlooking some
issue regarding 'always logging to stderr' part in non-task context.

-- 
- Lukas



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

Reply via email to