[pve-devel] [PATCH installer 0/3] add check/rename for already-existing ZFS rpool

Christoph Heiss c.heiss at proxmox.com
Mon Jul 8 13:27:17 CEST 2024


Ping. Series still applies cleanly.

On Thu, May 16, 2024 at 12:28:32PM GMT, Christoph Heiss wrote:
> Pretty straight forward overall, implements a check for an exising
> `rpool` on the system and ask the user whether they would like to rename
> it, much in the same way as it works for VGs already.
>
> Without this, the installer would silently create a second (and thus
> conflicting) `rpool` and cause a boot failure after the installation,
> since it does not know which pool to import exactly.
>
> Christoph Heiss (3):
>   proxmox: add zfs module for retrieving importable zpool info
>   low-level: install: split out random disk uid generation
>   low-level: install: check for already-existing `rpool` on install
>
>  Proxmox/Install.pm        | 47 ++++++++++++++++++++++++++++++++-----
>  Proxmox/Makefile          |  1 +
>  Proxmox/Sys/ZFS.pm        | 43 ++++++++++++++++++++++++++++++++++
>  test/Makefile             |  6 +++++
>  test/zfs-get-pool-list.pl | 49 +++++++++++++++++++++++++++++++++++++++
>  5 files changed, 140 insertions(+), 6 deletions(-)
>  create mode 100644 Proxmox/Sys/ZFS.pm
>  create mode 100755 test/zfs-get-pool-list.pl
>
> --
> 2.44.0
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>




More information about the pve-devel mailing list