(incubator-uniffle) branch master updated: [MINOR] fix(deploy): Fix invocation of kubernetes bash scripts (#1513)

[zuston]

This is an automated email from the ASF dual-hosted git repository.

zuston pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new 866f5ff12 [MINOR] fix(deploy): Fix invocation of kubernetes bash 
scripts (#1513)
866f5ff12 is described below

commit 866f5ff121948e2144729300a0f5cb03510db29d
Author: Enrico Minack <git...@enrico.minack.dev>
AuthorDate: Fri Feb 9 02:18:17 2024 +0100

    [MINOR] fix(deploy): Fix invocation of kubernetes bash scripts (#1513)
    
    ### What changes were proposed in this pull request?
    Do not invoke bash scripts via `sh ./SCRIPT.sh`, but `./SCRIPT.sh`.
    
    ### Why are the changes needed?
    These bash scripts are invoked via `sh ./SCRIPT.sh`, which overrides the 
shebang line in the script, effectively executing the scripts via `/bin/sh`. 
The scripts require `bash` though, so the invocation fails.
    
    ### Does this PR introduce _any_ user-facing change?
    No.
    
    ### How was this patch tested?
    Manually tested.
---
 deploy/kubernetes/integration-test/e2e/set-up-local-cluster.sh | 2 +-
 deploy/kubernetes/integration-test/e2e/start-e2e.sh            | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/deploy/kubernetes/integration-test/e2e/set-up-local-cluster.sh 
b/deploy/kubernetes/integration-test/e2e/set-up-local-cluster.sh
old mode 100644
new mode 100755
index 222c1745d..7e77476b9
--- a/deploy/kubernetes/integration-test/e2e/set-up-local-cluster.sh
+++ b/deploy/kubernetes/integration-test/e2e/set-up-local-cluster.sh
@@ -66,7 +66,7 @@ function prepare_local_k8s_cluster() {
 function build_rss_image() {
   cd $RSS_BUILD_DIR
   export IMAGE_VERSION=$TEST_VERSION
-  sh ./build.sh --registry $TEST_REGISTRY
+  ./build.sh --registry $TEST_REGISTRY
   cd "$OLDPWD"
 }
 
diff --git a/deploy/kubernetes/integration-test/e2e/start-e2e.sh 
b/deploy/kubernetes/integration-test/e2e/start-e2e.sh
index 5f2fdaa4d..dada47156 100755
--- a/deploy/kubernetes/integration-test/e2e/start-e2e.sh
+++ b/deploy/kubernetes/integration-test/e2e/start-e2e.sh
@@ -31,5 +31,5 @@ echo "BUILD_NEW_CLUSTER: $BUILD_NEW_CLUSTER"
 echo "BUILD_RSS_IMAGE: $BUILD_RSS_IMAGE"
 echo "BUILD_RSS_OPERATOR: $BUILD_RSS_OPERATOR"
 
-sh set-up-local-cluster.sh --registry "$TEST_REGISTRY" --build-kind-cluster 
"$BUILD_NEW_CLUSTER" \
+./set-up-local-cluster.sh --registry "$TEST_REGISTRY" --build-kind-cluster 
"$BUILD_NEW_CLUSTER" \
           --build-rss-image "$BUILD_RSS_IMAGE" --build-operator 
"$BUILD_RSS_OPERATOR"