-
Notifications
You must be signed in to change notification settings - Fork 92
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
Pass boolean properties to Redpanda configuration #472
Pass boolean properties to Redpanda configuration #472
Conversation
d3edd1c
to
1f8820d
Compare
59c79e3
to
fea0960
Compare
https://github.com/redpanda-data/helm-charts/actions/runs/4829528122/jobs/8604675668 Dedicated test that should be run only in origin not in fork is correctly detected by Github Action and it is cancelled. |
fea0960
to
f482f65
Compare
{{- $configurationBooleanProp := list "admin_api_require_auth" | ||
"aggregate_metrics" | ||
"auto_create_topics_enabled" | ||
"cloud_storage_disable_tls" | ||
"cloud_storage_enable_compacted_topic_reupload" | ||
"cloud_storage_enable_remote_read" | ||
"cloud_storage_enable_remote_write" | ||
"cloud_storage_enable_segment_merging" | ||
"cloud_storage_enabled" | ||
"disable_batch_cache" | ||
"disable_metrics" | ||
"disable_public_metrics" | ||
"enable_controller_log_rate_limiting" | ||
"enable_coproc" | ||
"enable_idempotence" | ||
"enable_leader_balancer" | ||
"enable_metrics_reporter" | ||
"enable_pid_file" | ||
"enable_rack_awareness" | ||
"enable_sasl" | ||
"enable_transactions" | ||
"enable_usage" | ||
"features_auto_enable" | ||
"kafka_enable_authorization" | ||
"kafka_enable_partition_reassignment" | ||
"kafka_qdc_enable" | ||
"legacy_group_offset_retention_enabled" | ||
"memory_abort_on_alloc_failure" | ||
"release_cache_on_segment_roll" | ||
"storage_strict_data_init" -}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The boolean properties was taken from running cluster v23.1.7 by the following command:
kubectl exec redpanda-0 -- curl -sk https://127.0.0.1:9644/v1/cluster_config/schema | jq -c '.properties | to_entries | .[] | select(.value.type | contains("boolean")) | .key'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated as 23.1.8 added new boolean use_fetch_scheduler_group
f482f65
to
5724ce7
Compare
@@ -71,7 +107,10 @@ data: | |||
{{- end }} | |||
{{- with (dig "cluster" dict .Values.config) }} | |||
{{- range $key, $element := .}} | |||
{{- if $element }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could do something like https://helm.sh/docs/chart_template_guide/function_list/#type-functions
how about {{ if or (eq (typeOf $element) boolean) $element }}
that way we do not have to know an exhaustive list of boolean configs. We leave the responsibility of the user to know what they are doing and we offload maintenance.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure I can change that.
0021525
to
93e93ef
Compare
93e93ef
to
3ad2c33
Compare
3ad2c33
to
762a63f
Compare
rebased |
fixes #470