You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openwhisk.apache.org by ra...@apache.org on 2018/01/11 16:05:23 UTC

[incubator-openwhisk-deploy-kube] branch master updated: fix Travis CI regression introduced by 84f6e38d. (#150)

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

rabbah pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-deploy-kube.git


The following commit(s) were added to refs/heads/master by this push:
     new 147d549  fix Travis CI regression introduced by 84f6e38d. (#150)
147d549 is described below

commit 147d5493e3eae887705b4f65dba800d204c83192
Author: David Grove <dg...@users.noreply.github.com>
AuthorDate: Thu Jan 11 11:05:21 2018 -0500

    fix Travis CI regression introduced by 84f6e38d. (#150)
    
    Only use `minikube ip` to get WSK_HOST when `kubectl describe nodes`
    returns "minikube" as the hostname.  When running with the none driver
    (like in travis CI), minikube ip returns local host, which works
    for the wsk CLI externally, but not for the wsk CLI when invoked
    from a Job running within minikube.
    
    Regression was not detected during PR testing because the fix
    of 63f96636 to check error conditions in the job's init.sh has not
    been merged to master when 84f6e38d was tested.
---
 tools/travis/build.sh | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/tools/travis/build.sh b/tools/travis/build.sh
index 9e112a2..d537b19 100755
--- a/tools/travis/build.sh
+++ b/tools/travis/build.sh
@@ -158,7 +158,10 @@ popd
 pushd kubernetes/ingress
   WSK_PORT=$(kubectl -n openwhisk describe service nginx | grep https-api | grep NodePort| awk '{print $3}' | cut -d'/' -f1)
   APIGW_PORT=$(kubectl -n openwhisk describe service apigateway | grep mgmt | grep NodePort| awk '{print $3}' | cut -d'/' -f1)
-  WSK_HOST=$(minikube ip)
+  WSK_HOST=$(kubectl describe nodes | grep Hostname: | awk '{print $2}')
+  if [ "$WSK_HOST" = "minikube" ]; then
+      WSK_HOST=$(minikube ip)
+  fi
   kubectl -n openwhisk create configmap whisk.ingress --from-literal=api_host=$WSK_HOST:$WSK_PORT --from-literal=apigw_url=http://$WSK_HOST:$APIGW_PORT
   wsk property set --auth `cat ../cluster-setup/auth.guest` --apihost $WSK_HOST:$WSK_PORT
 popd

-- 
To stop receiving notification emails like this one, please contact
['"commits@openwhisk.apache.org" <co...@openwhisk.apache.org>'].