[flink] branch release-1.8 updated: [FLINK-11971] Fix kubernetes check in end-to-end test

[jincheng]

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

jincheng pushed a commit to branch release-1.8
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/release-1.8 by this push:
     new eb57156  [FLINK-11971] Fix kubernetes check in end-to-end test
eb57156 is described below

commit eb571567ccbf5cb663e91cbbe3d9d8685b6b52fa
Author: Aljoscha Krettek <aljoscha.kret...@gmail.com>
AuthorDate: Wed Mar 20 16:20:05 2019 +0100

    [FLINK-11971] Fix kubernetes check in end-to-end test
    
    Before, this was using string parsing but the output of minikube status
    depends on the minikube version and the system.
    
    This now simply uses the status code.
    
    This closes #8024.
---
 flink-end-to-end-tests/test-scripts/test_kubernetes_embedded_job.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/flink-end-to-end-tests/test-scripts/test_kubernetes_embedded_job.sh 
b/flink-end-to-end-tests/test-scripts/test_kubernetes_embedded_job.sh
index 96320fd..d37deca 100755
--- a/flink-end-to-end-tests/test-scripts/test_kubernetes_embedded_job.sh
+++ b/flink-end-to-end-tests/test-scripts/test_kubernetes_embedded_job.sh
@@ -40,8 +40,7 @@ function cleanup {
 }
 
 function check_kubernetes_status {
-    local status=`minikube status`
-    echo ${status} | grep -q "minikube: Running cluster: Running kubectl: 
Correctly Configured"
+    minikube status
     return $?
 }
 
@@ -50,7 +49,8 @@ function start_kubernetes_if_not_running {
         minikube start
     fi
 
-    return $(check_kubernetes_status)
+    check_kubernetes_status
+    return $?
 }
 
 trap cleanup EXIT