JiniousChoi edited a comment on issue #446: Add support for external Kafka as 
an option
URL: 
https://github.com/apache/incubator-openwhisk-deploy-kube/issues/446#issuecomment-479702986
 
 
   For Nginx, by the way, then, are we going to deprecate/remove the Nginx pod 
that gets deployed by the Helm chart later? ever? My reasoning why I ask you so 
is as follows:
   
   I deployed the Openwhisk on a private k8s cluster with a ingress in front of 
it. And it seems that a simple wsk command `wsk list` doesn't use the nginx pod 
in the cluster since by judging the request header from `wsk -v list` the 
requesting path is like `/api/v1/...` which would be directed to the controller 
service on the k8s cluster by the ingress rules at 
https://github.com/apache/incubator-openwhisk-deploy-kube/blob/master/helm/openwhisk/templates/frontdoor-ingress.yaml#L25-#L50.
   So, if one installs an ingress in front, he can delete the 
`deployment.apps/owdev-nginx` if I got the mechanism right.
   
   So, in this sense, as the question I posed at the beginning, do you think, 
in the near future, we would be likely to remove the not-much-useful nginx pod 
from the Helm chart and guide the devops as on how to utilize the ingress?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to