Move 'LoadBalancer' section under pachd, rename to 'externalService' and make ports configurable

pachyderm / helmchart

Helm Chart for Pachyderm

5 stars 9 forks source link

Move 'LoadBalancer' section under pachd, rename to 'externalService' and make ports configurable #65

Closed chainlink closed 3 years ago

chainlink commented 3 years ago

Move the load balancer section under pachd in values.yaml as it only controls pachd
Rename to externalService instead of loadbalancer for consistency (controls a service object)
Make external port numbers configurable for this service - Folks might want to be able to access pachd on a custom external port

philwinder commented 3 years ago

Hi @chainlink . Just stumbled across this. My use case: I need to use nodeports for a demo, but the pach dash defaults to 30081, and there is no setting to change this.

My browser inspector suggests that it is configurable:

var port = this.paramsFor('application').port;

We just need to pass this in at the chart level.

chainlink commented 3 years ago

@philwinder just circling back here, I have a PR which makes the pach ports configurable, but Dash is typically accessed behind an ingress. Does that work for your use case?