[pve-devel] applied: [PATCH pve-manager] pve7to8: fix broken pipe warning for dkms status invocation
Thomas Lamprecht
t.lamprecht at proxmox.com
Fri Apr 4 12:50:22 CEST 2025
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 at proxmox.com>
> Signed-off-by: Christian Ebner <c.ebner at 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.
More information about the pve-devel
mailing list