[pbs-devel] [PATCH proxmox-backup 0/3] fix #3786: resync corrupt chunks in sync-job

Gabriel Goller g.goller at proxmox.com
Tue Oct 15 15:18:20 CEST 2024


Add an option `resync-corrupt` that resyncs corrupt snapshots when running
sync-job. This option checks if the local snapshot failed the last
verification and if it did, overwrites the local snapshot with the
remote one.

This is quite useful, as we currently don't have an option to "fix" 
broken chunks/snapshots in any way, even if a healthy version is on 
another (e.g. offsite) instance.

Important things to note are also: this has a slight performance 
penalty, as all the manifests have to be looked through, and a 
verification job has to be run beforehand, otherwise we do not know 
if the snapshot is healthy.

Note: This series was originally written by Shannon! I just picked it 
up, rebased, and fixed the obvious comments on the last series.

Changelog since RFC (Shannon's work):
 - rename option from deep-sync to resync-corrupt
 - rebase on latest master (and change implementation details, as a 
     lot has changed around sync-jobs)

proxmox-backup:

Gabriel Goller (3):
  fix #3786: api: add resync-corrupt option to sync jobs
  fix #3786: ui/cli: add resync-corrupt option on sync-jobs
  fix #3786: docs: add resync-corrupt option to sync-job

 docs/managing-remotes.rst         |  6 +++
 pbs-api-types/src/jobs.rs         | 11 +++++
 pbs-datastore/src/backup_info.rs  | 13 +++++-
 src/api2/config/sync.rs           |  4 ++
 src/api2/pull.rs                  |  9 +++-
 src/bin/proxmox-backup-manager.rs | 13 +++++-
 src/server/pull.rs                | 68 +++++++++++++++++++++----------
 www/window/SyncJobEdit.js         | 11 +++++
 8 files changed, 110 insertions(+), 25 deletions(-)


Summary over all repositories:
  8 files changed, 110 insertions(+), 25 deletions(-)

-- 
Generated by git-murpp 0.7.1




More information about the pbs-devel mailing list