[flink] branch master 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 master
in repository https://gitbox.apache.org/repos/asf/flink.git


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

commit 56d81e0dcadd07d634f92ac464a49d3313f56621
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