[pve-devel] [PATCH] fix #5869: d/postinst: do not fail if ifreload fails on first install

Stoiko Ivanov s.ivanov at proxmox.com
Wed Nov 13 20:01:13 CET 2024


erroring out here will not help getting a running network, therefore
just log a warning.

Tested the original reporters case (running in a debootstrap) with:
```
mmdebstrap \
 --variant=minbase \
 --include=/tmp/ifupdown2_3.2.0-1+pmx10_all.deb \
 --hook-dir=/usr/share/mmdebstrap/hooks/file-mirror-automount \
 bookworm \
 ./bookworm-amd64.tar.zst
```
ran through our install on plain debian guide:
https://pve.proxmox.com/wiki/Install_Proxmox_VE_on_Debian_12_Bookworm
with this package in a local repository to check for regressions

adding the reference to the commit that introduced the reload on
first-install, as this got shifted around a bit and it took me too
long to see that we did not want to error out here on purpose

Fixes: 86335f3f898c59864a09a3e13b8146c0a0ea079d
Signed-off-by: Stoiko Ivanov <s.ivanov at proxmox.com>
---
 debian/ifupdown2.postinst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/ifupdown2.postinst b/debian/ifupdown2.postinst
index 0137ea8..a22b825 100644
--- a/debian/ifupdown2.postinst
+++ b/debian/ifupdown2.postinst
@@ -114,7 +114,7 @@ case "$1" in
         if [ -f "/tmp/.ifupdown2-first-install" ] && [ ! -e /proxmox_install_mode ]; then
             proxmox_compatibility
             echo "Reloading network config on first install"
-            ifreload -a
+            ifreload -a || report_warn "Reloading failed"
             rm  /tmp/.ifupdown2-first-install
         fi
     ;;
-- 
2.39.5





More information about the pve-devel mailing list