From f3808e00c7fdbdea2efba5a2ad8499f792cfd4b2 Mon Sep 17 00:00:00 2001 From: ChrisBeaker Date: Tue, 16 Jul 2024 10:51:47 +0200 Subject: [PATCH] Fix: Add $SUDO prefix to transactional-update commands in install script This commit addresses an issue in the k3s install script where the transactional-update commands did not include the $SUDO variable. This omission caused the script to fail if sudo was not used. Adding the $SUDO prefix ensures that the script runs correctly in all environments. Signed-off-by: ChrisBeaker Signed-off-by: Derek Nola --- install.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/install.sh b/install.sh index b19f6cad85ee..84a6215a146d 100755 --- a/install.sh +++ b/install.sh @@ -669,7 +669,7 @@ EOF : "${INSTALL_K3S_SKIP_START:=true}" fi # create the /var/lib/rpm-state in SLE systems to fix the prein selinux macro - ${transactional_update_run} mkdir -p /var/lib/rpm-state + $SUDO ${transactional_update_run} mkdir -p /var/lib/rpm-state ;; coreos) rpm_installer="rpm-ostree --idempotent" @@ -941,7 +941,7 @@ elif type zypper >/dev/null 2>&1; then if [ "\${TRANSACTIONAL_UPDATE=false}" != "true" ] && [ -x /usr/sbin/transactional-update ]; then uninstall_cmd="transactional-update --no-selfupdate -d run \$uninstall_cmd" fi - \$uninstall_cmd + $SUDO \$uninstall_cmd rm -f /etc/zypp/repos.d/rancher-k3s-common*.repo fi EOF