Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Missing Configuration in bookkeeper.conf #23818

Open
3 tasks done
sandeep-mst opened this issue Jan 7, 2025 · 0 comments · May be fixed by cognitree/pulsar#18 or #23819
Open
3 tasks done

[Bug] Missing Configuration in bookkeeper.conf #23818

sandeep-mst opened this issue Jan 7, 2025 · 0 comments · May be fixed by cognitree/pulsar#18 or #23819
Labels
type/bug The PR fixed a bug or issue reported a bug

Comments

@sandeep-mst
Copy link

Search before asking

  • I searched in the issues and found nothing similar.

Read release policy

  • I understand that unsupported versions don't get bug fixes. I will attempt to reproduce the issue on a supported version of Pulsar client and Pulsar broker.

Version

OS - macOS 14.5 (23F79)
Java Version - 17
Pulsar Client Version - 4.0.1, 3.0.x
Pulsar Broker Version - 4.0.1, 3.0.x

Minimal reproduce step

  • Set env values for minorCompactionMaxTimeMillis and majorCompactionMaxTimeMillis and run apply-config-from-env.py.

Alternatively, it can be reproduced using pulsar-helm-chart and populating the values in .Values.bookkeeper.configData. Internally it uses the apply-config-from-env.py script to update values in bookkeeper.conf.

What did you expect to see?

minorCompactionMaxTimeMillis and majorCompactionMaxTimeMillis parameters being configured in bookkeeper.conf.

What did you see instead?

minorCompactionMaxTimeMillis and majorCompactionMaxTimeMillis parameters are missing from bookkeeper.conf even after running the script.

Anything else?

We need to add these parameters to bookkeeper.conf to allow the script to update them.

Are you willing to submit a PR?

  • I'm willing to submit a PR!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
1 participant