[pve-devel] applied: [PATCH pve-manager] pve7to8: fix broken pipe warning for dkms status invocation

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

Am 28.03.25 um 18:48 schrieb Christian Ebner:
> The check for dkms kernel modules relies on the output of `dkms
> status`. dkms command invocation will perform the following sanity
> check:
> ```
> if [ ! -e <(echo) ]; then
>     warn $"dkms will not function properly if /proc is not mounted."
> fi
> ```
> 
> This check will however throw the following warning when SIGPIPE is
> set to be ignored:
> 
> ```
> sbin/dkms: line 2497: echo: write error: Broken pipe
> ```
> 
> While only cosmetic, this can be confusing. Therefore, temporarily
> enable SIGPIPE before calling dkms, restoring the originally setting
> afterwards.
> 
> Reported-by: Alexander Zeidler <a.zeidler@proxmox.com>
> Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
> ---
>  PVE/CLI/pve7to8.pm | 3 +++
>  1 file changed, 3 insertions(+)
> 
>

applied, thanks!


Also backported this to stable-7 and bumped a new version there with
some small other fixes included that mostly help on upgrade; as while
PVE 7 is EOL in general, I'm fine with rolling out polishing to make
upgrade easier, as that takes away even more excuses for not upgrading
already.


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