[pbs-devel] applied-series: [PATCH-SERIES proxmox-backup/proxmox v3 00/10] replace print by log macro in libraries
Wolfgang Bumiller
w.bumiller at proxmox.com
Tue Jun 21 10:46:01 CEST 2022
applied series, thanks
On Wed, Jun 15, 2022 at 08:19:46AM +0000, Hannes Laimer wrote:
> This series mostly replaces print with the log macro in libs, it also replaces print
> in binaries where it is used to log stuff and not output the result of a command.
> In the process of replacing prints by log macros a few parameters controlling verbosity
> became obsolete and were removed(outputs that were behind an if(verbose).. are now log::debug),
> other 'verbose' parameters influenced the control flow and where therefore kept.
>
> The whole changes were split up into 7 seperate patches[2-8], this was done
> to aviod one huge patch file and improve readability. Those (maybe also 1)
> should be squashed when applied since they are not necesarilly buildable.
> The reason for that is that in a few places 'verbose' parameters were remove.
>
> A verion bump is also needed since patches 1 (and indirectly 2-8) depend on
> the function added to proxmox-router.
>
> v3, thanks @Wolfgang Bumiller <w.bumiller at proxmox.com>:
> * updated verbosity in some places
> * used log_enabled!(...)
> * removed unused 'print_info' function
> * rebased onto master
>
> * proxmox
> Hannes Laimer (1):
> router: add init_cli_logger helper function
>
> proxmox-router/Cargo.toml | 1 +
> proxmox-router/src/cli/mod.rs | 11 +++++++++++
> 2 files changed, 12 insertions(+)
>
> * proxmox-backup
> Hannes Laimer (10):
> bins: init cli logger
> pbs-client: replace print with log macro
> pbs-datastore: replace print with log macro
> pbs-fuse+pbs-tape: replace print with log macro
> proxmox-backup-client: replace print with log macro
> proxmox-file-restore: replace print with log macro
> proxmox-rest-server: replace print with log macro
> replace print with log macro
> docs: add note for setting verbosity level
> fixed_index: remove unused 'print_info' function
>
> docs/command-syntax.rst | 4 +
> examples/upload-speed.rs | 2 +-
> pbs-client/Cargo.toml | 1 +
> pbs-client/src/backup_writer.rs | 103 ++++++++----------
> pbs-client/src/catalog_shell.rs | 4 +-
> pbs-client/src/http_client.rs | 14 +--
> pbs-client/src/pxar/create.rs | 2 -
> pbs-client/src/pxar/extract.rs | 70 ++++--------
> pbs-client/src/pxar/fuse.rs | 18 ++-
> pbs-client/src/pxar/metadata.rs | 4 +-
> pbs-client/src/pxar_backup_stream.rs | 6 +-
> pbs-client/src/task_log.rs | 8 +-
> pbs-client/src/tools/key_source.rs | 2 +-
> pbs-datastore/src/catalog.rs | 6 +-
> pbs-datastore/src/datastore.rs | 2 +-
> pbs-datastore/src/dynamic_index.rs | 2 +-
> pbs-datastore/src/fixed_index.rs | 19 +---
> pbs-datastore/src/paperkey.rs | 2 +-
> pbs-fuse-loop/Cargo.toml | 3 +-
> pbs-fuse-loop/src/fuse_loop.rs | 12 +-
> pbs-tape/Cargo.toml | 1 +
> pbs-tape/src/bin/pmt.rs | 12 +-
> pbs-tape/src/bin/pmtx.rs | 10 +-
> pbs-tape/src/sg_pt_changer.rs | 4 +-
> proxmox-backup-client/Cargo.toml | 1 +
> proxmox-backup-client/src/benchmark.rs | 36 ++----
> proxmox-backup-client/src/catalog.rs | 6 +-
> proxmox-backup-client/src/key.rs | 27 +++--
> proxmox-backup-client/src/main.rs | 92 +++++++---------
> proxmox-backup-client/src/mount.rs | 12 +-
> proxmox-file-restore/Cargo.toml | 1 +
> proxmox-file-restore/src/block_driver_qemu.rs | 6 +-
> proxmox-file-restore/src/main.rs | 26 ++---
> proxmox-file-restore/src/qemu_helper.rs | 18 ++-
> proxmox-rest-server/src/api_config.rs | 4 +-
> proxmox-rest-server/src/command_socket.rs | 13 ++-
> proxmox-rest-server/src/file_logger.rs | 2 +-
> .../src/proxmox_restore_daemon/api.rs | 1 -
> pxar-bin/Cargo.toml | 1 +
> pxar-bin/src/main.rs | 51 +++------
> src/api2/admin/datastore.rs | 6 +-
> src/bin/proxmox-backup-debug.rs | 4 +-
> src/bin/proxmox-backup-manager.rs | 2 +
> src/bin/proxmox-tape.rs | 2 +
> src/bin/sg-tape-cmd.rs | 12 +-
> 45 files changed, 259 insertions(+), 375 deletions(-)
>
> --
> 2.30.2
More information about the pbs-devel
mailing list