* [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