Skip to content

Commit

Permalink
Increase max length of additionalConfig
Browse files Browse the repository at this point in the history
A maximum string lenth of 2,000 was too restrictive.
The new limit of 100,000 matches the limit of `advancedConfig`.

Resolves #1707
  • Loading branch information
ansd committed Aug 26, 2024
1 parent cfa0d03 commit 1383a10
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion api/v1beta1/rabbitmqcluster_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ type RabbitmqClusterConfigurationSpec struct {
// Modify to add to the rabbitmq.conf file in addition to default configurations set by the operator.
// Modifying this property on an existing RabbitmqCluster will trigger a StatefulSet rolling restart and will cause rabbitmq downtime.
// For more information on this config, see https://www.rabbitmq.com/configure.html#config-file
// +kubebuilder:validation:MaxLength:=2000
// +kubebuilder:validation:MaxLength:=100000
AdditionalConfig string `json:"additionalConfig,omitempty"`
// Specify any rabbitmq advanced.config configurations to apply to the cluster.
// For more information on advanced config, see https://www.rabbitmq.com/configure.html#advanced-config-file
Expand Down
2 changes: 1 addition & 1 deletion config/crd/bases/rabbitmq.com_rabbitmqclusters.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4886,7 +4886,7 @@ spec:
Modify to add to the rabbitmq.conf file in addition to default configurations set by the operator.
Modifying this property on an existing RabbitmqCluster will trigger a StatefulSet rolling restart and will cause rabbitmq downtime.
For more information on this config, see https://www.rabbitmq.com/configure.html#config-file
maxLength: 2000
maxLength: 100000
type: string
additionalPlugins:
description: 'List of plugins to enable in addition to essential plugins: rabbitmq_management, rabbitmq_prometheus, and rabbitmq_peer_discovery_k8s.'
Expand Down

0 comments on commit 1383a10

Please sign in to comment.