public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* Re: [pve-devel] [RFC pve-qemu] Add systemd journal logging patch
       [not found] <20200630120620.20176-1-s.reiter@proxmox.com>
@ 2020-09-03 16:30 ` Thomas Lamprecht
  2020-09-08 15:23 ` [pve-devel] applied: " Thomas Lamprecht
  1 sibling, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2020-09-03 16:30 UTC (permalink / raw)
  To: PVE development discussion, Stefan Reiter

On 30.06.20 14:06, Stefan Reiter wrote:
> Prints QEMU errors that occur *after* the "-daemonize" fork to the
> systemd journal, instead of pushing them into /dev/null like before.
> 
> Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>
> ---
> 
> Useful for debugging rust panics for example. I'm sure there's other ways to go
> about this (log files? pass the journal fd from outside? pipe it into the
> journal somehow?) but this one seems simple enough, though it of course requires
> linking QEMU against libsystemd.
> 
> @dietmar: is this similar to what you had in mind?
> 
>  debian/control                                |  1 +
>  ...ct-stderr-to-journal-when-daemonized.patch | 50 +++++++++++++++++++
>  debian/patches/series                         |  1 +
>  3 files changed, 52 insertions(+)
>  create mode 100644 debian/patches/pve/0052-PVE-redirect-stderr-to-journal-when-daemonized.patch
> 

any consens, updates regarding this?





^ permalink raw reply	[flat|nested] 2+ messages in thread

* [pve-devel] applied: [RFC pve-qemu] Add systemd journal logging patch
       [not found] <20200630120620.20176-1-s.reiter@proxmox.com>
  2020-09-03 16:30 ` [pve-devel] [RFC pve-qemu] Add systemd journal logging patch Thomas Lamprecht
@ 2020-09-08 15:23 ` Thomas Lamprecht
  1 sibling, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2020-09-08 15:23 UTC (permalink / raw)
  To: PVE development discussion, Stefan Reiter

On 30.06.20 14:06, Stefan Reiter wrote:
> Prints QEMU errors that occur *after* the "-daemonize" fork to the
> systemd journal, instead of pushing them into /dev/null like before.
> 
> Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>
> ---
> 
> Useful for debugging rust panics for example. I'm sure there's other ways to go
> about this (log files? pass the journal fd from outside? pipe it into the
> journal somehow?) but this one seems simple enough, though it of course requires
> linking QEMU against libsystemd.
> 
> @dietmar: is this similar to what you had in mind?
> 
>  debian/control                                |  1 +
>  ...ct-stderr-to-journal-when-daemonized.patch | 50 +++++++++++++++++++
>  debian/patches/series                         |  1 +
>  3 files changed, 52 insertions(+)
>  create mode 100644 debian/patches/pve/0052-PVE-redirect-stderr-to-journal-when-daemonized.patch
> 
>

applied, thanks!




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-09-08 15:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20200630120620.20176-1-s.reiter@proxmox.com>
2020-09-03 16:30 ` [pve-devel] [RFC pve-qemu] Add systemd journal logging patch Thomas Lamprecht
2020-09-08 15:23 ` [pve-devel] applied: " Thomas Lamprecht

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal