Skip to content

Commit 749bf91

Browse files
committed
Fix K3S_DATA_DIR when running as non-root user
When running the generated kill and uninstall scripts, the environment variable `K3S_DATA_DIR` got lost, because `sudo` by default doesn't preserve the environment. This is fixed by adding the `--preserve-env` flag. Signed-off-by: Reinhard Nägele <[email protected]>
1 parent 0144d9b commit 749bf91

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

install.sh

+5-5
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ get_pr_artifact_url() {
510510
if [ -z "${commit_id}" ]; then
511511
fatal "Installing PR builds requires GITHUB_TOKEN with k3s-io/k3s repo permissions"
512512
fi
513-
513+
514514
# GET request to the GitHub API to retrieve the Build workflow associated with the commit
515515
run_id=$(curl -s -H "Authorization: Bearer ${GITHUB_TOKEN}" "${github_api_url}/commits/${commit_id}/check-runs?check_name=build%20%2F%20Build" | jq -r '[.check_runs | sort_by(.id) | .[].details_url | split("/")[7]] | last')
516516

@@ -626,7 +626,7 @@ setup_selinux() {
626626
install_selinux_rpm ${rpm_site} ${rpm_channel} ${rpm_target} ${rpm_site_infix}
627627

628628
policy_error=fatal
629-
if [ "$INSTALL_K3S_SELINUX_WARN" = true ] || [ "${ID_LIKE:-}" = coreos ] ||
629+
if [ "$INSTALL_K3S_SELINUX_WARN" = true ] || [ "${ID_LIKE:-}" = coreos ] ||
630630
[ "${VARIANT_ID:-}" = coreos ] || [ "${VARIANT_ID:-}" = iot ]; then
631631
policy_error=warn
632632
fi
@@ -646,7 +646,7 @@ setup_selinux() {
646646
}
647647

648648
install_selinux_rpm() {
649-
if [ -r /etc/redhat-release ] || [ -r /etc/centos-release ] || [ -r /etc/oracle-release ] ||
649+
if [ -r /etc/redhat-release ] || [ -r /etc/centos-release ] || [ -r /etc/oracle-release ] ||
650650
[ -r /etc/fedora-release ] || [ -r /etc/system-release ] || [ "${ID_LIKE%%[ ]*}" = "suse" ]; then
651651
repodir=/etc/yum.repos.d
652652
if [ -d /etc/zypp/repos.d ]; then
@@ -775,7 +775,7 @@ create_killall() {
775775
info "Creating killall script ${KILLALL_K3S_SH}"
776776
$SUDO tee ${KILLALL_K3S_SH} >/dev/null << \EOF
777777
#!/bin/sh
778-
[ $(id -u) -eq 0 ] || exec sudo $0 $@
778+
[ $(id -u) -eq 0 ] || exec sudo --preserve-env=K3S_DATA_DIR $0 $@
779779
780780
K3S_DATA_DIR=${K3S_DATA_DIR:-/var/lib/rancher/k3s}
781781
@@ -877,7 +877,7 @@ create_uninstall() {
877877
$SUDO tee ${UNINSTALL_K3S_SH} >/dev/null << EOF
878878
#!/bin/sh
879879
set -x
880-
[ \$(id -u) -eq 0 ] || exec sudo \$0 \$@
880+
[ \$(id -u) -eq 0 ] || exec sudo --preserve-env=K3S_DATA_DIR \$0 \$@
881881
882882
K3S_DATA_DIR=\${K3S_DATA_DIR:-/var/lib/rancher/k3s}
883883

install.sh.sha256sum

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
d75e014f2d2ab5d30a318efa5c326f3b0b7596f194afcff90fa7a7a91166d5f7 install.sh
1+
73ca045f12174c87138d59dd14b23b8ab70d96c9eaead63104109d330daa8a6a install.sh

0 commit comments

Comments
 (0)