apache/pulsar-helm-chart
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Restore support for dbStorage_rocksDB_* settings defined in bookkeeper.configData (#580)

Browse Source
This commit is contained in:
Lari Hotari 2025-03-04 08:05:59 +02:00 committed by GitHub
parent 4bdf6d51eb
commit 5c1b7a9288
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
charts/pulsar/templates/bookkeeper-configmap.yaml
View File

@ -61,5 +61,19 @@ data:
{{- end }}
# TLS config
{{- include "pulsar.bookkeeper.config.tls" . | nindent 2 }}
{{- if .Values.bookkeeper.useRocksDBConfigInConfigData }}
# Set RocksDB default format version to 5
# RocksDB format_version 5 has been supported since RocksDB 6.6 . It's required for certain performance optimizations.
PULSAR_PREFIX_dbStorage_rocksDB_format_version: "5"
# Specify non-existing files to avoid Bookkeeper from loading RocksDB config from existing files
PULSAR_PREFIX_defaultRocksdbConf: "conf/non_existing_default_rocksdb.conf"
PULSAR_PREFIX_entryLocationRocksdbConf: "conf/non_existing_entry_location_rocksdb.conf"
PULSAR_PREFIX_ledgerMetadataRocksdbConf: "conf/non_existing_ledger_metadata_rocksdb.conf"
{{- else }}
# Specify existing files to load RocksDB config from existing files
PULSAR_PREFIX_defaultRocksdbConf: "conf/default_rocksdb.conf"
PULSAR_PREFIX_entryLocationRocksdbConf: "conf/entry_location_rocksdb.conf"
PULSAR_PREFIX_ledgerMetadataRocksdbConf: "conf/ledger_metadata_rocksdb.conf"
{{- end }}
{{ toYaml .Values.bookkeeper.configData | indent 2 }}
{{- end }}

4
charts/pulsar/values.yaml
View File

@ -728,6 +728,10 @@ bookkeeper:
## templates/bookkeeper-service-account.yaml
service_account:
annotations: {}
## Use RocksDB config in configData
## Use dbStorage_rocksDB_* / PULSAR_PREFIX_dbStorage_rocksDB_* settings defined in configData instead of conf/*_rocksdb.conf files in the Pulsar docker image
## See https://github.com/apache/bookkeeper/pull/3523 as reference
useRocksDBConfigInConfigData: true
## Bookkeeper configmap
## templates/bookkeeper-configmap.yaml
##