all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update
@ 2024-12-09 10:47 Gabriel Goller
  2024-12-09 12:08 ` Fiona Ebner
  2025-01-14  8:46 ` Gabriel Goller
  0 siblings, 2 replies; 5+ messages in thread
From: Gabriel Goller @ 2024-12-09 10:47 UTC (permalink / raw)
  To: pbs-devel

Previously we just wrote to syslog directly. This doesn't work anymore
since the tracing update and we won't get any output in the tasklog.

Reported-by: https://forum.proxmox.com/threads/pbs-3-3-0-dly-update-status-unknown.158764/
Reported-by: Fiona Ebner <f.ebner@proxmox.com>
Signed-off-by: Gabriel Goller <g.goller@proxmox.com>
---

Note: this patch relies on
https://lore.proxmox.com/pbs-devel/20241209104606.263045-1-g.goller@proxmox.com/T/#m0b852d2987004eaa42715e445efec0aba98aeafe

 src/bin/proxmox-daily-update.rs | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/bin/proxmox-daily-update.rs b/src/bin/proxmox-daily-update.rs
index dbfee07b4cd4..8e9cd8035cb3 100644
--- a/src/bin/proxmox-daily-update.rs
+++ b/src/bin/proxmox-daily-update.rs
@@ -109,14 +109,7 @@ async fn run(rpcenv: &mut dyn RpcEnvironment) -> Result<(), Error> {
 
 fn main() {
     proxmox_backup::tools::setup_safe_path_env();
-
-    if let Err(err) = syslog::init(
-        syslog::Facility::LOG_DAEMON,
-        log::LevelFilter::Info,
-        Some("proxmox-daily-update"),
-    ) {
-        eprintln!("unable to initialize syslog - {err}");
-    }
+    proxmox_log::init::journald_and_tasklog("PBS_LOG", proxmox_log::LevelFilter::INFO).expect("Failed to setup proxmox_log logger");
 
     let mut rpcenv = CliEnvironment::new();
     rpcenv.set_auth_id(Some(String::from("root@pam")));
-- 
2.39.5



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


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

* Re: [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update
  2024-12-09 10:47 [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update Gabriel Goller
@ 2024-12-09 12:08 ` Fiona Ebner
  2025-01-14  8:46 ` Gabriel Goller
  1 sibling, 0 replies; 5+ messages in thread
From: Fiona Ebner @ 2024-12-09 12:08 UTC (permalink / raw)
  To: Gabriel Goller, pbs-devel

Am 09.12.24 um 11:47 schrieb Gabriel Goller:
> Previously we just wrote to syslog directly. This doesn't work anymore
> since the tracing update and we won't get any output in the tasklog.
> 
> Reported-by: https://forum.proxmox.com/threads/pbs-3-3-0-dly-update-status-unknown.158764/

Note that you can shorten this to
https://forum.proxmox.com/threads/158764/
which is nicer for commit messages.

> Reported-by: Fiona Ebner <f.ebner@proxmox.com>

No need for that tag, I just pointed you to the forum thread :)


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


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

* Re: [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update
  2024-12-09 10:47 [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update Gabriel Goller
  2024-12-09 12:08 ` Fiona Ebner
@ 2025-01-14  8:46 ` Gabriel Goller
  2025-02-11 19:14   ` Thomas Lamprecht
  1 sibling, 1 reply; 5+ messages in thread
From: Gabriel Goller @ 2025-01-14  8:46 UTC (permalink / raw)
  To: pbs-devel

bump


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


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

* Re: [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update
  2025-01-14  8:46 ` Gabriel Goller
@ 2025-02-11 19:14   ` Thomas Lamprecht
  2025-02-13 10:13     ` Gabriel Goller
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Lamprecht @ 2025-02-11 19:14 UTC (permalink / raw)
  To: Proxmox Backup Server development discussion, Gabriel Goller

Am 14.01.25 um 09:46 schrieb Gabriel Goller:
> bump

It would be IMO better to wait for bumping until the dependencies are through,
it's a bit confusing otherwise.

You could send a v2 with Fiona's suggestion to the commit messages trailers
addressed once that happened.


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


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

* Re: [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update
  2025-02-11 19:14   ` Thomas Lamprecht
@ 2025-02-13 10:13     ` Gabriel Goller
  0 siblings, 0 replies; 5+ messages in thread
From: Gabriel Goller @ 2025-02-13 10:13 UTC (permalink / raw)
  To: Thomas Lamprecht; +Cc: Proxmox Backup Server development discussion

Oops, my bad, didn't notice the other patch wasn't merged yet, sorry.


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


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

end of thread, other threads:[~2025-02-13 10:13 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-12-09 10:47 [pbs-devel] [PATCH proxmox-backup] log: update to tracing in proxmox-daily-update Gabriel Goller
2024-12-09 12:08 ` Fiona Ebner
2025-01-14  8:46 ` Gabriel Goller
2025-02-11 19:14   ` Thomas Lamprecht
2025-02-13 10:13     ` Gabriel Goller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal