Re: [pbs-devel] [PATCH proxmox-backup v3] docs: replace concrete device paths with pseudo paths

[Lukas Wagner <l.wagner@proxmox.com>]

Looks good to me. Consider this:

Reviewed-by: Lukas Wagner <l.wagner@proxmox.com>

On 7/11/23 09:56, Philipp Hufnagl wrote:
> Having commonly used device paths (like /dev/sdb) in an example command
> may cause damage if the user simply copies them without checking. With a
> pseudo device path (like /dev/sdX), they would simply get an error
> 
> Signed-off-by: Philipp Hufnagl <p.hufnagl@proxmox.com>
> ---
>   docs/storage.rst | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/docs/storage.rst b/docs/storage.rst
> index f473adbe..60cf268e 100644
> --- a/docs/storage.rst
> +++ b/docs/storage.rst
> @@ -48,12 +48,12 @@ You can create an ``ext4`` or ``xfs`` filesystem on a disk using ``fs
>   create``, or by navigating to **Administration -> Storage/Disks -> Directory**
>   in the web interface and creating one from there. The following command creates
>   an ``ext4`` filesystem and passes the ``--add-datastore`` parameter, in order to
> -automatically create a datastore on the disk (in this case ``sdd``). This will
> +automatically create a datastore on the disk. This will
>   create a datastore at the location ``/mnt/datastore/store1``:
>   
>   .. code-block:: console
>   
> -  # proxmox-backup-manager disk fs create store1 --disk sdd --filesystem ext4 --add-datastore true
> +  # proxmox-backup-manager disk fs create store1 --disk sdX --filesystem ext4 --add-datastore true
>   
>   .. image:: images/screenshots/pbs-gui-disks-zfs-create.png
>     :align: right
> @@ -61,12 +61,12 @@ create a datastore at the location ``/mnt/datastore/store1``:
>   
>   You can also create a ``zpool`` with various raid levels from **Administration
>   -> Storage/Disks -> ZFS** in the web interface, or by using ``zpool create``. The command
> -below creates a mirrored ``zpool`` using two disks (``sdb`` & ``sdc``) and
> +below creates a mirrored ``zpool`` using two disks and
>   mounts it under ``/mnt/datastore/zpool1``:
>   
>   .. code-block:: console
>   
> -  # proxmox-backup-manager disk zpool create zpool1 --devices sdb,sdc --raidlevel mirror
> +  # proxmox-backup-manager disk zpool create zpool1 --devices sdX,sdY --raidlevel mirror
>   
>   .. note:: You can also pass the ``--add-datastore`` parameter here, to automatically
>     create a datastore from the disk.

-- 
- Lukas