all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: "Gabriel Goller" <g.goller@proxmox.com>
To: "Proxmox Backup Server development discussion"
	<pbs-devel@lists.proxmox.com>
Subject: Re: [pbs-devel] [PATCH proxmox-backup v9 14/26] docs: add removable datastores section
Date: Wed, 24 Apr 2024 13:53:16 +0200	[thread overview]
Message-ID: <D0SBYWQAJBCK.214I6OL1MK92@proxmox.com> (raw)
In-Reply-To: <20240423083000.28002-15-h.laimer@proxmox.com>

On Tue Apr 23, 2024 at 10:29 AM CEST, Hannes Laimer wrote:
> Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
> ---
>  docs/storage.rst | 29 +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
>
> diff --git a/docs/storage.rst b/docs/storage.rst
> index 4444c423..7bfd016c 100644
> --- a/docs/storage.rst
> +++ b/docs/storage.rst
> @@ -165,6 +165,35 @@ following command creates a new datastore called ``store1`` on
>    # proxmox-backup-manager datastore create store1 /backup/disk1/store1
>  
>  
> +Removable Datastores
> +^^^^^^^^^^^^^^^^^^^^
> +Removable datastores have a ``backing-device`` associadted with them, they can be

'associadted' -> 'associated'

> +mounted and unmounted. Other than that they behave the same way a normal Datastore

'Datastores' -> 'datastores'

> +would.
> +
> +They can be created on already correctly formatted partitions, as with normal Datastores

'Datastores' -> 'datastores'

> +the partitions should be either ``ext4`` or ``xfs``. It is also possible to

I think this sentence can be rewritten to be more legible as such: 

    They can be created on already correctly formatted partitions, which, 
    as with normal datastores, should be either ``ext4`` or ``xfs``.

> +create them on completely unused disks through "Administartion" > "Disks / Storage" > "Directory",

'Administartion' -> 'Administration'

> +using this methos the disk will be partitioned and formated actomatically for the datastore.

'methos' -> 'method'
'formated' -> 'formatted'
'actomatically' -> 'automatically'

> +
> +They are mounted automatically when plugged in. It is possible to create a
> +removable Datastore on one PBS and use it on mutiple, the device just has to be added

'Datastore' -> 'datastore'
'mutiple' -> 'multiple'

> +on all as a removable Datastore. This can be done using the normal `Create Datastore`

'Datastore' -> 'datastore'

> +form, by checking **Removable datastore**, selecting the device and setting the correct path.
> +If the devcice already contains a datastore at the speciefied path it'll just be added as

'devcice' -> 'device'
'speciefied' -> 'specified'

> +a new Datastore to the PBS instance and will be mounted whenever plugged in. Unmounting has

'Datastore' -> 'datastore'

> +to be done through the UI by clicking "Unmount" on the summary page or using the CLI
> +
> +.. code-block:: console
> +
> +  # proxmox-backup-manager datastore unmount store1
> +
> +both will wait for any running tasks and unmount the device once the last finishes.

I think this reads better:

    both will wait for any running tasks to finish and unmount the device.

> +
> +All removable datastores will be mounted under ``/mnt/datastore/<name>`` and the
> +specified path is the on-device path.

This part of the sentence is kind of unclear, could you maybe explain it
more simply?

    and the specified path is the on-device path

> +
> +
>  Managing Datastores
>  ^^^^^^^^^^^^^^^^^^^
>  



_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


  reply	other threads:[~2024-04-24 11:53 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-23  8:29 [pbs-devel] [PATCH proxmox-backup v9 00/26] add removable datastores Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 01/26] tools: add disks utility functions Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 02/26] config: factor out method to get the absolute datastore path Hannes Laimer
2024-04-24 13:26   ` Wolfgang Bumiller
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 03/26] pbs-api-types: add backing-device to DataStoreConfig Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 04/26] maintenance: add 'Unmount' maintenance type Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 05/26] disks: add UUID to partition info Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 06/26] datastore: add helper for checking if a removable datastore is available Hannes Laimer
2024-04-24 11:16   ` Gabriel Goller
2024-04-24 13:30   ` Wolfgang Bumiller
2024-04-25  6:05     ` Hannes Laimer
2024-04-25  7:19       ` Wolfgang Bumiller
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 07/26] api: admin: add (un)mount endpoint for removable datastores Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 08/26] api: removable datastore creation Hannes Laimer
2024-04-24 13:04   ` Wolfgang Bumiller
2024-04-25  8:52     ` Wolfgang Bumiller
2024-04-25  8:58       ` Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 09/26] api: disks list: add exclude-used flag Hannes Laimer
2024-04-24 13:09   ` Wolfgang Bumiller
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 10/26] pbs-api-types: add removable/is-available flag to DataStoreListItem Hannes Laimer
2024-04-24 13:13   ` Wolfgang Bumiller
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 11/26] bin: manager: add (un)mount command Hannes Laimer
2024-04-24 11:25   ` Gabriel Goller
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 12/26] add auto-mounting for removable datastores Hannes Laimer
2024-04-24 13:18   ` Wolfgang Bumiller
2024-04-25  5:32     ` Hannes Laimer
2024-04-25  5:34       ` Hannes Laimer
2024-04-24 13:35   ` Wolfgang Bumiller
2024-04-25  4:59     ` Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 13/26] datastore: handle deletion of removable datastore properly Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 14/26] docs: add removable datastores section Hannes Laimer
2024-04-24 11:53   ` Gabriel Goller [this message]
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 15/26] ui: add partition selector form Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 16/26] ui: add removable datastore creation support Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 17/26] ui: add (un)mount button to summary Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 18/26] ui: tree: render unmounted datastores correctly Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 19/26] ui: utils: make parseMaintenanceMode more robust Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 20/26] ui: add datastore status mask for unmounted removable datastores Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 21/26] ui: maintenance: fix disable msg field if no type is selected Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 22/26] ui: render 'unmount' maintenance mode correctly Hannes Laimer
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 23/26] api: node: allow creation of removable datastore through directory endpoint Hannes Laimer
2024-04-24 11:56   ` Gabriel Goller
2024-04-24 13:21   ` Wolfgang Bumiller
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 24/26] api: node: include removable datastores in directory list Hannes Laimer
2024-04-24 13:06   ` Wolfgang Bumiller
2024-04-23  8:29 ` [pbs-devel] [PATCH proxmox-backup v9 25/26] node: disks: replace BASE_MOUNT_DIR with DATASTORE_MOUNT_DIR Hannes Laimer
2024-04-23  8:30 ` [pbs-devel] [PATCH proxmox-backup v9 26/26] ui: support create removable datastore through directory creation Hannes Laimer

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=D0SBYWQAJBCK.214I6OL1MK92@proxmox.com \
    --to=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 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