search

temporalio / helm-charts

Temporal Helm charts
MIT License
282 stars 316 forks source link

[Feature Request] Add ingress for temporal server frontend #430

Closed debugger24 closed 2 weeks ago

debugger24 commented 9 months ago

Is your feature request related to a problem? Please describe.

There are scenarios where we need to expose temporal FE with ingress, currently I am adding extending this helm chart to add that support. Would be nice if we can add it.

Describe the solution you'd like

Add ingress for temporal FE just like web

Additional context

I can pick this up if we think we should add it.

joshbranham commented 9 months ago

Yeah it looks like the frontend: only supports service right now, which we are using an NLB.

debugger24 commented 9 months ago

Yes, both FE and Web(UI) has service, we should expose both through ingress

Note: FE != UI

joshbranham commented 9 months ago

I would think that is a useful change to add 👌