From: Dominik Csapak <d.csapak@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox-backup v2 0/8] tape: single snapshot restore
Date: Wed, 5 May 2021 12:09:10 +0200 [thread overview]
Message-ID: <20210505100918.506-1-d.csapak@proxmox.com> (raw)
v2 of the series, some small parts are still in-progress/unfinished:
* api path is (imho) not optimal, but did not find something better
(integration into existing restore call gets ugly fast...)
* schema for the snapshot list is not done yet (but not hard..)
* gui for multiselection is not done yet (have to think about how
we do that for a good ux)
questions still to answer:
do we really want to have the ability to restore multiple 'single' snapshots
in one go? if not, it would drastically simplify the code
changes from v1:
* use parallel handler for chunk restore
* rebase on master
* add patch to return manifest from try_restore_snapshot_archive
* using of Arc<WorkerTask> like we do now in rest of the file
@Dietmar, could you test on real tape hardware if it works correctly as-is?
Dominik Csapak (8):
tape/drive: add 'move_to_file' to TapeDriver trait
tape/media_catalog: add helpers to look for snapshot/chunk files
api2/tape/restore: factor out check_datastore_privs
api2/tape/restore: remove unnecessary params from
(try_)restore_snapshot_archive
api2/tape/restore: return backup manifest in
try_restore_snapshot_archive
api2/tape/restore: add 'restore-single' api path
bin/proxmox-tape: add restore-single command to proxmox-tape
ui: tape: add single snapshot restore
src/api2/tape/mod.rs | 1 +
src/api2/tape/restore.rs | 668 +++++++++++++++++++++++++++++++--
src/bin/proxmox-tape.rs | 62 +++
src/tape/drive/lto/mod.rs | 4 +
src/tape/drive/mod.rs | 3 +
src/tape/drive/virtual_tape.rs | 22 ++
src/tape/media_catalog.rs | 20 +
www/tape/BackupOverview.js | 41 ++
www/tape/window/TapeRestore.js | 26 ++
9 files changed, 821 insertions(+), 26 deletions(-)
--
2.20.1
next reply other threads:[~2021-05-05 10:09 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-05 10:09 Dominik Csapak [this message]
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 1/8] tape/drive: add 'move_to_file' to TapeDriver trait Dominik Csapak
2021-05-06 5:56 ` [pbs-devel] applied: " Dietmar Maurer
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 2/8] tape/media_catalog: add helpers to look for snapshot/chunk files Dominik Csapak
2021-05-06 5:59 ` [pbs-devel] applied: " Dietmar Maurer
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 3/8] api2/tape/restore: factor out check_datastore_privs Dominik Csapak
2021-05-06 6:01 ` [pbs-devel] applied: " Dietmar Maurer
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 4/8] api2/tape/restore: remove unnecessary params from (try_)restore_snapshot_archive Dominik Csapak
2021-05-06 6:02 ` [pbs-devel] applied: " Dietmar Maurer
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 5/8] api2/tape/restore: return backup manifest in try_restore_snapshot_archive Dominik Csapak
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 6/8] api2/tape/restore: add 'restore-single' api path Dominik Csapak
2021-05-05 10:53 ` Thomas Lamprecht
2021-05-05 12:48 ` Dominik Csapak
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 7/8] bin/proxmox-tape: add restore-single command to proxmox-tape Dominik Csapak
2021-05-05 11:04 ` Thomas Lamprecht
2021-05-05 12:50 ` Dominik Csapak
2021-05-05 10:09 ` [pbs-devel] [PATCH proxmox-backup v2 8/8] ui: tape: add single snapshot restore Dominik Csapak
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=20210505100918.506-1-d.csapak@proxmox.com \
--to=d.csapak@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 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