Skip to content
This repository was archived by the owner on Mar 26, 2025. It is now read-only.

Commit 5582f1d

Browse files
committed
Fix panic when brokerId is greater than zero
1 parent 2705265 commit 5582f1d

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

pkg/resources/kafka/configmap.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,8 +175,14 @@ func getInternalListeners(iListeners []v1beta1.InternalListenerConfig, id int32,
175175

176176
func (r Reconciler) generateBrokerConfig(id int32, brokerConfig *v1beta1.BrokerConfig, superUsers []string, loadBalancerIP string, log logr.Logger) string {
177177
parsedReadOnlyClusterConfig := util.ParsePropertiesFormat(r.KafkaCluster.Spec.ReadOnlyConfig)
178+
var parsedReadOnlyBrokerConfig = map[string]string{}
178179

179-
parsedReadOnlyBrokerConfig := util.ParsePropertiesFormat(r.KafkaCluster.Spec.Brokers[id].ReadOnlyConfig)
180+
for _, broker := range r.KafkaCluster.Spec.Brokers {
181+
if broker.Id == id {
182+
parsedReadOnlyBrokerConfig = util.ParsePropertiesFormat(broker.ReadOnlyConfig)
183+
break
184+
}
185+
}
180186

181187
if err := mergo.Merge(&parsedReadOnlyBrokerConfig, parsedReadOnlyClusterConfig); err != nil {
182188
log.Error(err, "error occurred during merging readonly configs")

0 commit comments

Comments
 (0)