[pbs-devel] [PATCH proxmox-backup 1/4] cli: proxmox-tape: fix ns/depth parameter

[pbs-devel] [PATCH proxmox-backup 1/4] cli: proxmox-tape: fix ns/depth parameter

[Dominik Csapak]

was forgotten after recent rebase

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 src/bin/proxmox-tape.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/bin/proxmox-tape.rs b/src/bin/proxmox-tape.rs
index 80094d25..af39556e 100644
--- a/src/bin/proxmox-tape.rs
+++ b/src/bin/proxmox-tape.rs
@@ -841,11 +841,11 @@ async fn clean_drive(mut param: Value) -> Result<(), Error> {
                 schema: GROUP_FILTER_LIST_SCHEMA,
                 optional: true,
             },
-            namespace: {
+            ns: {
                 type: BackupNamespace,
                 optional: true,
             },
-            "recursion-depth": {
+            "max-depth": {
                 schema: NS_MAX_DEPTH_SCHEMA,
                 optional: true,
             },
-- 
2.30.2

[pbs-devel] [PATCH proxmox-backup 2/4] docs: tape: remove note about global content namespace

[Dominik Csapak]

that is actually not true, we save the datastore in the chunk archives
as well as the snapshot archives, otherwise we could not backup
multiple datastores to a single media-set.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 docs/tape-backup.rst | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/docs/tape-backup.rst b/docs/tape-backup.rst
index bfb7a35b..fa0172a0 100644
--- a/docs/tape-backup.rst
+++ b/docs/tape-backup.rst
@@ -565,13 +565,6 @@ a single media pool, so a job only uses tapes from that pool.
    the password. Please make sure to remember the password, in case
    you need to restore the key.
 
-
-.. NOTE:: We use global content namespace, meaning we do not store the
-   source datastore name. Because of this, it is impossible to distinguish
-   store1:/vm/100 from store2:/vm/100. Please use different media pools
-   if the sources are from different namespaces with conflicting names
-   (for example, if the sources are from different Proxmox VE clusters).
-
 .. image:: images/screenshots/pbs-gui-tape-pools-add.png
   :align: right
   :alt: Tape Backup: Add a media pool
-- 
2.30.2

[pbs-devel] [PATCH proxmox-backup 3/4] docs: tape/restore: mention single snapshot restore

[Dominik Csapak]

what it is, how to use it and the caveats

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 docs/tape-backup.rst | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/docs/tape-backup.rst b/docs/tape-backup.rst
index fa0172a0..0eb32412 100644
--- a/docs/tape-backup.rst
+++ b/docs/tape-backup.rst
@@ -831,6 +831,29 @@ data disk (datastore):
 
  # proxmox-tape restore 9da37a55-aac7-4deb-91c6-482b3b675f30 mystore
 
+Single Snapshot Restore
+^^^^^^^^^^^^^^^^^^^^^^^
+
+Sometimes it is not necessary to restore a whole media-set, but only some
+specific snapshots from the tape. This can be achieved  with the ``snapshots``
+parameter:
+
+
+.. code-block:: console
+
+ // proxmox-tape restore <media-set-uuid> <datastore> [<snapshot>]
+
+ # proxmox-tape restore 9da37a55-aac7-4deb-91c6-482b3b675f30 mystore sourcestore:host/hostname/2022-01-01T00:01:00Z
+
+This first restores the snapshot to a temporary location, then restores the relevant
+chunk archives, and finally restores the snapshot data to the target datastore.
+
+The ``snapshot`` parameter can be given multiple times, so one can restore
+multiple snapshots with one restore action.
+
+.. NOTE:: When using the single snapshot restore, the tape must be traversed
+   more than once, which, if you restore many snapshots at once, can take longer
+   than restoring the whole datastore.
 
 Update Inventory
 ~~~~~~~~~~~~~~~~
-- 
2.30.2

[pbs-devel] [PATCH proxmox-backup 4/4] docs: tape: add information about namespaces

[Dominik Csapak]

which are backed up, how to use the new parameters and how to map
them during restore

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 docs/tape-backup.rst | 44 ++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/docs/tape-backup.rst b/docs/tape-backup.rst
index 0eb32412..8888dacb 100644
--- a/docs/tape-backup.rst
+++ b/docs/tape-backup.rst
@@ -681,6 +681,16 @@ To remove a job, please use:
 
  # proxmox-tape backup-job remove job2
 
+By default, all (recursive) namespaces of the datastore are included in a tape
+backup. You can specify a single namespace wth ``ns`` and a depth with
+``max-depth``. For example:
+
+.. code-block:: console
+
+ # proxmox-tape backup-job update job2 --ns mynamespace --max-depth 3
+
+If no `max-depth` is given, it will include all recursive namespaces.
+
 .. image:: images/screenshots/pbs-gui-tape-backup-jobs-add.png
   :align: right
   :alt: Tape Backup: Add a backup job
@@ -797,6 +807,16 @@ The following options are available:
   media set into import-export slots. The operator can then pick up
   those tapes and move them to a media vault.
 
+--ns  The namespace to backup.
+
+  If you only want to backup a specific namespace. If omitted, the root
+  namespaces is assumed.
+
+--max-depth  The depth to recurse namespaces.
+
+  ``0`` means no recursion at all (only the given namespace). If omitted,
+  all namespaces are recursed (below the the given one).
+
 
 Restore from Tape
 ~~~~~~~~~~~~~~~~~
@@ -855,6 +875,30 @@ multiple snapshots with one restore action.
    more than once, which, if you restore many snapshots at once, can take longer
    than restoring the whole datastore.
 
+Namespaces
+^^^^^^^^^^
+
+It is also possible to select and map specific namespaces from a media-set
+during a restore. This is possible with the ``namespaces`` parameter.
+The format of the parameter is
+
+.. code-block:: console
+
+ store=<source-datastore>[,source=<source-ns>][,target=<target-ns>][,max-depth=<depth>]
+
+If ``source`` or ``target`` is not given, the root namespace is assumed.
+When no ``max-depth`` is given, the source namespace will be fully recursed.
+
+An example restore command:
+
+.. code-block:: console
+
+ # proxmox-tape restore 9da37a55-aac7-4deb-91c6-482b3b675f30 mystore --namespaces store=sourcedatastore,source=ns1,target=ns2,max-depth=2
+
+The parameter can be given multiple times. It can also be combined with the
+``snapshots`` parameter to only restore those snapshots and map them to different
+namespaces.
+
 Update Inventory
 ~~~~~~~~~~~~~~~~
 
-- 
2.30.2

[pbs-devel] applied-series: [PATCH proxmox-backup 1/4] cli: proxmox-tape: fix ns/depth parameter

[Thomas Lamprecht]

Am 5/13/22 um 15:39 schrieb Dominik Csapak:
> was forgotten after recent rebase
> 
> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
> ---
>  src/bin/proxmox-tape.rs | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
>

applied series, thanks!