[GitHub] [apisix-helm-chart] cr7258 commented on pull request #401: fix: enable tls for the tcp address

[GitBox <gi...@apache.org>]

cr7258 commented on PR #401:
URL: https://github.com/apache/apisix-helm-chart/pull/401#issuecomment-1321380703

   I know this feature was implemented in [this PR](https://github.com/apache/apisix-ingress-controller/pull/1051) for apisix ingress, but apisix doesn't support to enable tls in helm chart now,  we only can set tcp port in the `stream_proxy.tcp` field in the values.yaml file.
   ```yaml
   apisix:
     stream_proxy:
       tcp:
         -  9100
         -  9200
   ```
   if we want to use this feature in apisix ingress, we must enable tls in apisix first. [So we should have the helm templates support to enable tls](https://github.com/apache/apisix-helm-chart/pull/401/files).  like this:
   ```yaml
   apisix:
     stream_proxy:
       tcp:
         - addr: 9100
           tls: true  # enable tls for tcp 9100
         - 9200
   ```


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