From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id DA766694D8 for ; Fri, 11 Mar 2022 20:06:06 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 2F9D62803A for ; Fri, 11 Mar 2022 20:06:06 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id 31EFF27EDA for ; Fri, 11 Mar 2022 20:06:01 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id 012A746E39 for ; Fri, 11 Mar 2022 20:06:01 +0100 (CET) From: Stoiko Ivanov To: pve-devel@lists.proxmox.com Date: Fri, 11 Mar 2022 20:05:38 +0100 Message-Id: <20220311190548.2937377-1-s.ivanov@proxmox.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.213 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record T_SCC_BODY_TEXT_LINE -0.01 - Subject: [pve-devel] [PATCH pve-kernel-meta/pve-installer v3] improve boot-mode switching X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Mar 2022 19:06:06 -0000 v2->v3: * rebased on master * added Thomas' suggestion of not clobbering an existing /etc/kernel/proxmox-boot-manual-kernels * fixed a typo in promox-boot/functions (proxmox-boot-boot-manual-kernels) * gave the installer another succesful try (with btrfs raid1) original cover-letter for the v2: these patches are still based on v3 of the pin/unpin addtion [1]. Not sending the pin/unpin again, since the feedback was mostly positive and I think Thomas had already some improvements on top of it prepared locally. changes v1->v2: * added a fix for #3729 to pve-kernel-meta - not directly related to the series - but since the series moves the affected lines around quite a bit I thought we can merge them together (can gladly send it on top of master) * else mostly incorporated Fabian's feedback - huge thanks! * comments where appropriate are added to the individual patches original cover-letter for the v1: This patchset is the result from the discussion on the rfc for using grub as only boot loader [0]. The patches for pve-kernel-meta are based on top of the v3 of the `pin/unpin` addition [1]. The patches for pve-installer are independent from the pve-kernel-meta ones (my tests used the current PVE 7.1 ISO w/o updating the pve-kernel-helper package) - but I still consider them linked. changes from the rfc: * instead of using grub everywhere - we now use systemd-boot always for efi systems and leave grub only for legacy booted systems * instead of unconditionally installing the boot-loaders for both modes on all ESPs users now can opt-in to installing the boot-loader for the other bootmode (both are updated if their configs are present, by default we only install the boot-loader for the current boot mode) Tested the pve-kernel-meta with 3 VMs (ZFS+uefi, ZFS+legacy, ext4+uefi) and switching boot-modes (after the necessary preparations). Tested the pve-installer by scping proxinstall into a VM and installing on btrfs+uefi, ext4+uefi, btrfs+legacy [0] https://lists.proxmox.com/pipermail/pve-devel/2022-February/051562.html [1] https://lists.proxmox.com/pipermail/pve-devel/2022-February/051732.html pve-kernel-meta: Stoiko Ivanov (8): rename pve-efiboot-manual-kernels to proxmox-boot-manual-kernels proxmox-boot: add reinit subcommand proxmox-boot: add helpers to check for bootloader configs proxmox-boot: refresh based on bootloader config instead of bootmode proxmox-boot: remove now obsolete EFI/proxmoxdir if it exists proxmox-boot: init bootloaders based on esp contents proxmox-boot: allow to manually specify bootloader proxmox-boot: fix #3729 add --graceful to bootctl invocation bin/grub-install-wrapper | 19 +----- bin/proxmox-boot-tool | 107 ++++++++++++++++++++++++------ debian/pve-kernel-helper.postinst | 4 ++ proxmox-boot/functions | 18 ++++- proxmox-boot/zz-proxmox-boot | 84 +++++++++-------------- 5 files changed, 143 insertions(+), 89 deletions(-) pve-installer: Stoiko Ivanov (2): remove /mnt/hostrun after install use proxmox-boot-tool for all uefi installs proxinstall | 81 ++++++++++------------------------------------------- 1 file changed, 15 insertions(+), 66 deletions(-) -- 2.30.2