ibm-messaging / mq-helm

Apache License 2.0
28 stars 35 forks source link

Variables for port numbers #25

Closed JGrzybowski closed 2 years ago

JGrzybowski commented 2 years ago

Is there a reason that port numbers are not customizable in the chart? Eg. in load balancer? If not, can I provide a pull request allowing such feature?

callumpjackson commented 2 years ago

I'm assuming you are suggesting that the load balancer services are changed to have a:

port: <loadbalancer port>
targetPort: <target container port>

Where the <loadbalancer port> can be customized within the helm chart? If so, happy for you to create a pull request with this change, but please assure that you read and explicitly say in the pull request that you accept the CLA.

callumpjackson commented 2 years ago

Closing as no additional action required at this time.