From 019b0afdd830e36f5e4b7a5b237e6a4339e6a6be Mon Sep 17 00:00:00 2001 From: ChrisBeaker Date: Tue, 13 Aug 2024 00:28:36 +0200 Subject: [PATCH] Fix: Add $SUDO prefix to transactional-update commands in install script (#10531) * 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 ++-- install.sh.sha256sum | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/install.sh b/install.sh index b19f6cad85..84a6215a14 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 diff --git a/install.sh.sha256sum b/install.sh.sha256sum index 7a767af64c..32a040820d 100644 --- a/install.sh.sha256sum +++ b/install.sh.sha256sum @@ -1 +1 @@ -10adf00df73b55e10b0e17dbf423f3b2707b8540c90cfa8b07c34f9688c23a30 install.sh +68ae766b4c4fd2c8032f712cdd346c8f9d1ae60eaeb4736a1155effda44ea23d install.sh