You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/09/03 21:04:35 UTC

[GitHub] [pulsar-helm-chart] frankjkelly opened a new issue #158: Helm chart does not support Istio: Add support for Istio standard port naming

frankjkelly opened a new issue #158:
URL: https://github.com/apache/pulsar-helm-chart/issues/158


   **Describe the bug**
   
   Istio expects port names to abide by a certain convention
   https://istio.io/latest/docs/ops/configuration/traffic-management/protocol-selection/#explicit-protocol-selection
   
   ```
   This can be configured in two ways:
   
   By the name of the port: name: <protocol>[-<suffix>].
   In Kubernetes 1.18+, by the appProtocol field: appProtocol: <protocol>.
   The following protocols are supported:
   
   http
   http2
   https
   tcp
   tls
   grpc
   grpc-web
   mongo
   mysql*
   redis*
   udp (UDP will not be proxied, but the port can be explicitly declared as UDP)
   ```
   
   
   
   
   **To Reproduce**
   Pulsar ports are named as 
   
   `bookie`
   https://github.com/apache/pulsar-helm-chart/blob/master/charts/pulsar/templates/bookkeeper-service.yaml#L35-L36
   `pulsar`
   https://github.com/apache/pulsar-helm-chart/blob/master/charts/pulsar/templates/broker-service.yaml#L37-L38
   `pulsarssl`
   https://github.com/apache/pulsar-helm-chart/blob/master/charts/pulsar/templates/broker-service.yaml#L43-L44
   and several other examples
   
   **Expected behavior**
   Pulsar Helm chart supports Istio port naming standard
   
   


-- 
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: dev-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar-helm-chart] frankjkelly commented on issue #158: Helm chart does not support Istio: Add support for Istio standard port naming

Posted by GitBox <gi...@apache.org>.
frankjkelly commented on issue #158:
URL: https://github.com/apache/pulsar-helm-chart/issues/158#issuecomment-942538854


   Fixed by https://github.com/apache/pulsar-helm-chart/pull/162 


-- 
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: dev-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar-helm-chart] frankjkelly closed issue #158: Helm chart does not support Istio: Add support for Istio standard port naming

Posted by GitBox <gi...@apache.org>.
frankjkelly closed issue #158:
URL: https://github.com/apache/pulsar-helm-chart/issues/158


   


-- 
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: dev-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar-helm-chart] frankjkelly commented on issue #158: Helm chart does not support Istio: Add support for Istio standard port naming

Posted by GitBox <gi...@apache.org>.
frankjkelly commented on issue #158:
URL: https://github.com/apache/pulsar-helm-chart/issues/158#issuecomment-942538854


   Fixed by https://github.com/apache/pulsar-helm-chart/pull/162 


-- 
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: dev-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar-helm-chart] frankjkelly closed issue #158: Helm chart does not support Istio: Add support for Istio standard port naming

Posted by GitBox <gi...@apache.org>.
frankjkelly closed issue #158:
URL: https://github.com/apache/pulsar-helm-chart/issues/158


   


-- 
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: dev-unsubscribe@pulsar.apache.org

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