From 6a48a7c929f394aa4ca9ccc4e809f112beae9513 Mon Sep 17 00:00:00 2001 From: Thomas Stallinger Date: Mon, 20 Jul 2026 14:51:22 +0200 Subject: [PATCH] fix: tolerate nmcli connection.filename failure for in-memory live connections export_connection_profile() crashed the whole installer under set -e/ pipefail when the active NetworkManager connection has no backing keyfile (common for ephemeral live-boot DHCP connections) - nmcli returned a non-zero exit status even though the guard below already handles an empty result gracefully. Every other nmcli call in this function already has this || true safety net; this one was missing it. Found via a live end-to-end boot test in Proxmox. --- scripts/modules/05_network.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/modules/05_network.sh b/scripts/modules/05_network.sh index 3e0286b..4b3acd9 100755 --- a/scripts/modules/05_network.sh +++ b/scripts/modules/05_network.sh @@ -440,7 +440,7 @@ export_connection_profile() { "${NMCLI}" \ --get-values connection.filename \ connection show "${connection_name}" 2>/dev/null | - head -n 1 + head -n 1 || true )" if [[ -z "${source_file}" || ! -f "${source_file}" ]]; then