Merge pull request #486 from alexander-demicev/fixpluginupdate

🐛 Use absolute path in update scripts
kubernetes-sigs · Apr 9, 2024 · d72fb20 · d72fb20
2 parents ac43057 + ec7a7aa
commit d72fb20
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/hack/update-helm-repo.sh b/hack/update-helm-repo.sh
@@ -3,6 +3,8 @@
 set -o errexit
 set -o pipefail
 
-REPO_ROOT=$(dirname "${BASH_SOURCE}")/..
+# Resolve the absolute path of the directory containing the script
+SCRIPT_DIR=$(realpath "$(dirname "${BASH_SOURCE[0]}")")
+REPO_ROOT="$SCRIPT_DIR/.."
 
 cd $REPO_ROOT/hack/chart-update; go run . -release-tag=$1; cd -
diff --git a/hack/update-plugin-yaml.sh b/hack/update-plugin-yaml.sh
@@ -3,6 +3,8 @@
 set -o errexit
 set -o pipefail
 
-REPO_ROOT=$(dirname "${BASH_SOURCE}")/..
+# Resolve the absolute path of the directory containing the script
+SCRIPT_DIR=$(realpath "$(dirname "${BASH_SOURCE[0]}")")
+REPO_ROOT="$SCRIPT_DIR/.."
 
-docker run --rm -v $REPO_ROOT:/home/app ghcr.io/rajatjindal/krew-release-bot:v0.0.46 krew-release-bot template --tag $1 --template-file .krew.yaml > $REPO_ROOT/plugins/clusterctl-operator.yaml
+docker run --rm -v "$REPO_ROOT":/home/app ghcr.io/rajatjindal/krew-release-bot:v0.0.46 krew-release-bot template --tag "$1" --template-file .krew.yaml > "$REPO_ROOT"/plugins/clusterctl-operator.yaml