From 2390a17faa75f7926033184777e65ee24eb0d119 Mon Sep 17 00:00:00 2001
From: Daniel Hilst <392820+dhilst@users.noreply.github.com>
Date: Tue, 1 Sep 2026 18:45:55 -0300
Subject: [PATCH] fix(configeth): netplan leaves the NIC down during
 provisioning

configeth's netplan arm took the interface down on every reconfigure, but the
`netplan_apply` that brings it back is gated on reboot_nic_bool. During the
diskful provision postscripts stage reboot_nic_bool is 0, so the link went down
and stayed down until the node rebooted -- and it is the install NIC the
postscripts are still talking over.

Gate the down on reboot_nic_bool as well, so the two halves are paired the way
the redhat arm already pairs them. Nothing is lost by skipping it: the drop-in
is written either way and takes effect at boot, which is the same reasoning the
existing comment gives for skipping the apply.

The ifupdown arm is deliberately left alone -- it answers its `ifdown` with an
unconditional `ifup`, so it never leaves the link down.

Signed-off-by: Daniel Hilst <392820+dhilst@users.noreply.github.com>
---
 xCAT/postscripts/configeth | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/xCAT/postscripts/configeth b/xCAT/postscripts/configeth
index c54883581..b036dc6ab 100755
--- a/xCAT/postscripts/configeth
+++ b/xCAT/postscripts/configeth
@@ -1598,7 +1598,12 @@ else
         if [ $bool_restart_flag -eq 1 ];then
             if [ "$str_nic_status" = "up" ];then
                 if [ "$str_os_type" = "debian" ] && [ "$netplan_active" = "1" ];then
-                    ip link set dev $str_nic_name down > /dev/null 2>/dev/null
+                    #the netplan_apply that brings the link back is gated on reboot_nic_bool,
+                    #so taking it down here ungated leaves the NIC down until the next reboot
+                    #in the provision postscripts stage -- over the NIC we are talking on.
+                    if [ $reboot_nic_bool -eq 1 ]; then
+                        ip link set dev $str_nic_name down > /dev/null 2>/dev/null
+                    fi
                 elif [ "$str_os_type" = "debian" ];then
                     ifdown --force $str_nic_name > /dev/null
                 else