You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/02/07 10:59:53 UTC

[GitHub] [apisix-helm-chart] tokers commented on issue #224: missing TCP port after set stream tcp port on values file

tokers commented on issue #224:
URL: https://github.com/apache/apisix-helm-chart/issues/224#issuecomment-1031337436


   > > @labaq The `ports` field in the Pod definition is just used to tell readers or other controllers that this pod will expose these ports, but it won't affect the real behaviors of the process inside the pod, ports missing from the field can still be listened.
   > > But I think it would be better to advertise all ports in the `ports` field, would you like to submit a PR to enhance this? Thanks!
   > 
   > @tokers
   > 
   > I think the service of apisix-gateway should expose the `stream proxy tcp/udp port` on `spec.ports`. Otherwise, these ports cannot be accessed through the service. I will try to fix it.
   
   Yes, that's true for `Service`.


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

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org