-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
[bitnami/kafka] add support for helm parameter maps for extraConfig #25646
Comments
Thank you for bringing this issue to our attention. We appreciate your involvement! If you're interested in contributing a solution, we welcome you to create a pull request. The Bitnami team is excited to review your submission and offer feedback. You can find the contributing guidelines here. Your contribution will greatly benefit the community. Feel free to reach out if you have any questions or need assistance. |
Use a yaml parameter map instead of a flat file for the extraConfig parameter. This allows for easier overloading both between the default and broker / controller values; and between multiple values.yaml.
Use a yaml parameter map instead of a flat file for the extraConfig parameter. This allows for easier overloading both between the default and broker / controller values; and between multiple values.yaml. Signed-off-by: Ruben Van Wanzeele <rubenvw@ngdata.com>
Hi @carrodher, I created a PR to add this. Let me know what you think. |
Use a yaml parameter map instead of a flat file for the extraConfig parameter. This allows for easier overloading both between the default and broker / controller values; and between multiple values.yaml. Signed-off-by: Ruben Van Wanzeele <rubenvw@ngdata.com>
Thank you for opening this issue and submitting the associated Pull Request. Our team will review and provide feedback. Once the PR is merged, the issue will automatically close. Your contribution is greatly appreciated! |
Name and Version
bitnami/kafka 28.1.1
What is the problem this feature will solve?
With the migration to Kafka 3.5, an
extraConfig
parameter was added to the chart.We regret that this is just a text value instead of a helm parameter map.
This choice blocks the overloading of a certain setting via additional values.yaml layers.
With the current implementation, you need to duplicate all
extraConfig
if you want to change a single parameter in it.As an example, instead of this
we would prefer to have
to allow overloading.
What is the feature you are proposing to solve the problem?
helm parameter maps to allow overloading the extraConfig.
The text was updated successfully, but these errors were encountered: