Skip to content

Commit

Permalink
Fix wrong kfserving url (#2551)
Browse files Browse the repository at this point in the history
* Fix wrong kfserving url

* Add comment about inferenceservice url format
  • Loading branch information
hougangliu authored and k8s-ci-robot committed Nov 6, 2019
1 parent 7e37145 commit 94c9eaa
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion components/kubeflow/kfserving/src/kfservingdeployer.py
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,9 @@ def deploy_model(action, model_name, default_model_uri, canary_model_uri, canary
print('Sample test commands: ')
print('# Note: If Istio Ingress gateway is not served with LoadBalancer, use $CLUSTER_NODE_IP:31380 as the ISTIO_INGRESS_ENDPOINT')
print('ISTIO_INGRESS_ENDPOINT=$(kubectl -n istio-system get service istio-ingressgateway -o jsonpath=\'{.status.loadBalancer.ingress[0].ip}\')')
print('curl -X GET -H "Host: ' + url.sub('', model_status['status']['url']) + '" $ISTIO_INGRESS_ENDPOINT')
# model_status['status']['url'] is like http://flowers-sample.kubeflow.example.com/v1/models/flowers-sample
host, path = url.sub('', model_status['status']['url']).split("/", 1)
print('curl -X GET -H "Host: ' + host + '" http://$ISTIO_INGRESS_ENDPOINT/' + path)
except:
print('Model is not ready, check the logs for the Knative URL status.')
if not os.path.exists(os.path.dirname(output_path)):
Expand Down

0 comments on commit 94c9eaa

Please sign in to comment.