Skip to content

Commit fad422d

Browse files
committed
fix(kafka): add Xms and Xmx JVM defaults
1 parent eca35d4 commit fad422d

File tree

3 files changed

+10
-1
lines changed

3 files changed

+10
-1
lines changed

charts/radar-kafka/README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ Consult the [documentation](https://github.com/lsst-sqre/strimzi-registry-operat
6060
| strimzi-registry-operator | object | `{"clusterName":"radar","operatorNamespace":"default"}` | Values for schema registry operator ref: https://github.com/lsst-sqre/strimzi-registry-operator |
6161
| strimzi-registry-operator.clusterName | string | `"radar"` | Keep in sync with 'fullnameOverride' |
6262
| strimzi-registry-operator.operatorNamespace | string | `"default"` | Keep in sync with namespace used by deployment |
63-
| kafka | object | `{"insyncReplicas":2,"metadataVersion":"3.9-IV0","partitions":9,"podSecurityContext":{},"replicas":3,"replicationFactor":3,"resources":{"limits":{"cpu":1,"memory":"2Gi"},"requests":{"cpu":"100m","memory":"1Gi"}},"securityContext":{},"storage":{"size":"10Gi"}}` | Values for Kafka cluster deployed by Strimzi kafka operator |
63+
| kafka | object | `{"insyncReplicas":2,"javaOptions":{"Xms":"512m","Xmx":"512m"},"metadataVersion":"3.9-IV0","partitions":9,"podSecurityContext":{},"replicas":3,"replicationFactor":3,"resources":{"limits":{"cpu":1,"memory":"2Gi"},"requests":{"cpu":"100m","memory":"1Gi"}},"securityContext":{},"storage":{"size":"10Gi"}}` | Values for Kafka cluster deployed by Strimzi kafka operator |
6464
| kafka.replicas | int | `3` | Number of Kafka brokers |
6565
| kafka.replicationFactor | int | `3` | Number of replicas for Kafka topics |
6666
| kafka.insyncReplicas | int | `2` | Number of in-sync kafka broker replicas |
@@ -69,6 +69,7 @@ Consult the [documentation](https://github.com/lsst-sqre/strimzi-registry-operat
6969
| kafka.storage | object | `{"size":"10Gi"}` | Storage size for Kafka pods |
7070
| kafka.resources.requests | object | `{"cpu":"100m","memory":"1Gi"}` | Resource requests for Kafka pods |
7171
| kafka.resources.limits | object | `{"cpu":1,"memory":"2Gi"}` | Resource limits for Kafka pods |
72+
| kafka.javaOptions | object | `{"Xms":"512m","Xmx":"512m"}` | JVM options for Kafka pods |
7273
| kafka.podSecurityContext | object | `{}` | Security Context for Kafka pods ref: https://strimzi.io/docs/operators/latest/deploying#assembly-security-providers-str |
7374
| kafka.securityContext | object | `{}` | Security Context for Kafka containers ref: https://strimzi.io/docs/operators/latest/deploying#assembly-security-providers-str |
7475
| schema-registry | object | check `values.yaml` | Values for schema registry deployed by strimzi-registry-operator ref: https://github.com/lsst-sqre/strimzi-registry-operator |

charts/radar-kafka/templates/kafka.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@ spec:
2525
default.replication.factor: {{ $topicReplicationFactor }}
2626
min.insync.replicas: {{ $insyncReplicas }}
2727
num.partitions: {{ $topicPartitions }}
28+
{{- if .Values.kafka.javaOptions }}
29+
jvmOptions: {{ .Values.kafka.javaOptions | toYaml | nindent 6 }}
30+
{{- end }}
2831
listeners:
2932
- name: plain
3033
port: 9092

charts/radar-kafka/values.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,11 @@ kafka:
7777
cpu: 1
7878
memory: 2Gi
7979

80+
# -- JVM options for Kafka pods
81+
javaOptions:
82+
Xms: 512m
83+
Xmx: 512m
84+
8085
# -- Security Context for Kafka pods
8186
# ref: https://strimzi.io/docs/operators/latest/deploying#assembly-security-providers-str
8287
podSecurityContext: {}

0 commit comments

Comments
 (0)