From: Wolfgang Bumiller <w.bumiller@proxmox.com>
To: Gabriel Goller <g.goller@proxmox.com>
Cc: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] applied-series: [PATCH proxmox{, -backup, -perl-rs} 0/4] Introduce builder for logging initialization
Date: Wed, 19 Mar 2025 12:30:09 +0100 [thread overview]
Message-ID: <mvo74klkqzoaq6gnlui7qmtbkid5v2gkhg4joejlpd6ve7pdlb@dpwyq6dhy4nl> (raw)
In-Reply-To: <20250218161641.563080-1-g.goller@proxmox.com>
applied series, but needed to fix up some doc tests
also factored the no-worker-task filter-fn out into a zst
On Tue, Feb 18, 2025 at 05:16:37PM +0100, Gabriel Goller wrote:
> To move away from product-specific logging functions or overly long and
> descriptive initialization functions, add a builder-type structure.
> This structure will combine layers and compose a logger more descriptively.
>
> The is backwards compatible as the old functions have not been removed and are
> only deprecated.
>
> This is the first version because the approach compared to the last series
> changed quite a lot.
> Previous series: https://lore.proxmox.com/pbs-devel/20241209104606.263045-1-g.goller@proxmox.com/
>
> proxmox:
>
> Gabriel Goller (2):
> log: introduce logging builder
> log: add layer for pve workertasks in perlmod crates
>
> proxmox-log/src/builder.rs | 141 ++++++++++++++++++++++++++
> proxmox-log/src/lib.rs | 93 +++++++----------
> proxmox-log/src/pve_task_formatter.rs | 31 ++++++
> 3 files changed, 209 insertions(+), 56 deletions(-)
> create mode 100644 proxmox-log/src/builder.rs
> create mode 100644 proxmox-log/src/pve_task_formatter.rs
>
>
> proxmox-backup:
>
> Gabriel Goller (1):
> log: use new builder initializer
>
> pbs-tape/src/bin/pmt.rs | 5 +++--
> pbs-tape/src/bin/pmtx.rs | 5 +++--
> proxmox-backup-client/src/main.rs | 6 ++++--
> proxmox-file-restore/src/main.rs | 7 +++++--
> pxar-bin/src/main.rs | 7 +++++--
> src/bin/proxmox-backup-api.rs | 6 ++++--
> src/bin/proxmox-backup-debug.rs | 6 ++++--
> src/bin/proxmox-backup-manager.rs | 8 ++++++--
> src/bin/proxmox-backup-proxy.rs | 6 ++++--
> src/bin/proxmox-daily-update.rs | 13 ++++++-------
> src/bin/proxmox-tape.rs | 6 ++++--
> src/bin/sg-tape-cmd.rs | 5 +++--
> 12 files changed, 51 insertions(+), 29 deletions(-)
>
>
> proxmox-perl-rs:
>
> Gabriel Goller (1):
> log: use new logging builder, print to stderr and journald
>
> common/src/logger.rs | 7 ++++++-
> pmg-rs/Cargo.toml | 1 -
> pmg-rs/src/lib.rs | 2 +-
> pve-rs/Cargo.toml | 1 -
> 4 files changed, 7 insertions(+), 4 deletions(-)
>
>
> Summary over all repositories:
> 19 files changed, 267 insertions(+), 89 deletions(-)
>
> --
> Generated by git-murpp 0.8.0
>
>
> _______________________________________________
> pbs-devel mailing list
> pbs-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
>
>
_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
prev parent reply other threads:[~2025-03-19 11:30 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-18 16:16 [pbs-devel] " Gabriel Goller
2025-02-18 16:16 ` [pbs-devel] [PATCH proxmox 1/2] log: introduce logging builder Gabriel Goller
2025-02-18 16:16 ` [pbs-devel] [PATCH proxmox 2/2] log: add layer for pve workertasks in perlmod crates Gabriel Goller
2025-02-18 16:16 ` [pbs-devel] [PATCH proxmox-backup 1/1] log: use new builder initializer Gabriel Goller
2025-02-18 16:16 ` [pbs-devel] [PATCH proxmox-perl-rs 1/1] log: use new logging builder, print to stderr and journald Gabriel Goller
2025-03-19 11:30 ` Wolfgang Bumiller [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=mvo74klkqzoaq6gnlui7qmtbkid5v2gkhg4joejlpd6ve7pdlb@dpwyq6dhy4nl \
--to=w.bumiller@proxmox.com \
--cc=g.goller@proxmox.com \
--cc=pbs-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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