redpanda-data / helm-charts

Redpanda Helm Chart
http://redpanda.com
Apache License 2.0
62 stars 92 forks source link

🫐 🐛 Operator: Allow to configure multiple advertised address:port pairs for external listeners #1150

Open c4milo opened 3 months ago

c4milo commented 3 months ago

What happened?

Currently, we can only configure N external ports, not addresses. Supporting configuring multiple address:port listeners and address:port advertised listeners is a requirement to be able to implement private links through a single network load balancer.

How can we reproduce it (as minimally and precisely as possible)?. Please include values file.

```console $ helm get values -n --all # paste output here ```

Anything else we need to know?

No response

Which are the affected charts?

Redpanda, Operator

Chart Version(s)

```console $ helm -n list # paste output here ```

Cloud provider

JIRA Link: K8S-143

bpraseed commented 3 months ago

@mattschumpert - as we discussed

mattschumpert commented 3 months ago

@chrisseto

mattschumpert commented 3 months ago

I think this ticket should be in the Operator repo @c4milo

chrisseto commented 3 months ago

@c4milo don't we need to support something dynamic in the advertised_port field? I don't see how that would be handled in your example.

RafalKorepta commented 3 months ago

@c4milo As far as I remember operator implementation that enables Private Links you would like to see per Redpanda Pod dedicated port. As helm chart uses StastefulSet all Redpanda Pods are identical, so all container ports would be open per Redpanda Pod (same as operator implementation), you can create unique Pods with your definition. It can only follow @paulzhang97 implementation.

REF

RafalKorepta commented 2 months ago

Waiting for https://github.com/redpanda-data/helm-charts/pull/1191