[pve-devel] applied: [PATCH kernel-helper/manager v3] check for fitting grub-meta package on uefi systems
Thomas Lamprecht
t.lamprecht at proxmox.com
Mon Nov 6 18:13:43 CET 2023
Am 11/10/2023 um 15:23 schrieb Stoiko Ivanov:
> v2->v3:
> * adapted Friedrich's feedback (huge thanks for the patience and attention
> to semantically important details!!) - so that the pve7to8 warning is
> actually understandable
>
> v1->v2:
> * adapted Friedrich's feedback (huge thanks!)
> ** fixed the wrongly negated check for installed grub-efi-amd64 in the
> boot-tool hook.
> ** Rephrased the error-message in pve7to8 to 2 sentences. I tried adding a
> newline as well, however this results in the message not being printed
> in the warning color anymore (most likely due to [0]) - and I felt this
> to be more important than having it on a separate line.
>
> [0] https://perldoc.perl.org/Term::ANSIColor#RESTRICTIONS
>
> original cover-letter for v1:
> The following patchset is a followup to the one for the installer:
> https://lists.proxmox.com/pipermail/pve-devel/2023-September/059270.html
>
> As suggested by Thomas - adding the check to proxmox-kernel-helper seems
> like a good idea. While adding it to d/postinst I thought that this might
> not be the best place - and that getting the warning upon every
> kernel-upgrade would be better vs. upon every upgrade of
> proxmox-kernel-helper (which are far less often).
> (Can gladly send the version with d/postinst as well)
>
> If the pve-manager patch gets applied - I'd push the equivalent change to
> pmg and provide one for pbs.
>
> Tested on legacy and uefi VMs installed with pve-8.0 iso and
> grub-efi-amd64 (and systemd-boot) removed vs. installed.
>
> proxmox-kernel-helper
> Stoiko Ivanov (2):
> proxmox-boot-tool: do not exit early in kernel-hook
> proxmox-boot-tool: check if correct grub metapackage is installed
>
> src/proxmox-boot/zz-proxmox-boot | 21 ++++++++++++++++++++-
> 1 file changed, 20 insertions(+), 1 deletion(-)
>
> pve-manager:
> Stoiko Ivanov (1):
> pve7to8: check for proper grub meta-package for bootmode
>
> PVE/CLI/pve7to8.pm | 39 +++++++++++++++++++++++----------------
> 1 file changed, 23 insertions(+), 16 deletions(-)
>
applied, with Friedrich's R-b/T-b tags, thanks!
I did some rewording for the kernel-helper, aligning it a bit more with the
one from pve6to7
More information about the pve-devel
mailing list