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>'].