search

apache / pulsar-helm-chart

Official Apache Pulsar Helm Chart
https://pulsar.apache.org/
Apache License 2.0
212 stars 224 forks source link

feat: add loadBalancerClass for proxy and pulsar-manager #546

Closed lenglet-k closed 2 weeks ago

lenglet-k commented 2 weeks ago

Fixes #545

Motivation

Add loadBalancerClass for use metallb loadbalancer service on private kubernetes cluster

Modifications

Add loadbalancerClass on proxy-service.yaml and pulsar-manager-service.yaml

Verifying this change

lhotari commented 2 weeks ago

k8s docs for loadBalancerClass: https://kubernetes.io/docs/concepts/services-networking/service/#load-balancer-class

lenglet-k commented 1 week ago

Hello, @lhotari thanks for fix. Possible to have a 3.7.1 with this improvement ?

lhotari commented 1 week ago

Hello, @lhotari thanks for fix. Possible to have a 3.7.1 with this improvement ?

@lenglet-k There's some overhead in the Apache release process so I'd rather batch more changes in the next release. In Apache Software Foundation (ASF), there are some policies around releases and that's why it cannot happen purely by automation and it requires release voting. On your end, it's worth using a fork of the helm chart for your deployments so that you don't have to wait for upstream releases of the Apache Pulsar helm chart in order to start using some improvements.