Skip to content

Expose Kafka Connect jvmOptions via chart values#355

Open
clintonium-119 wants to merge 2 commits into
masterfrom
fix-kafka-heap-jvm-options
Open

Expose Kafka Connect jvmOptions via chart values#355
clintonium-119 wants to merge 2 commits into
masterfrom
fix-kafka-heap-jvm-options

Conversation

@clintonium-119
Copy link
Copy Markdown
Member

@clintonium-119 clintonium-119 commented May 7, 2026

Summary

  • Plumb .Values.jvmOptions through to KafkaConnect.spec.jvmOptions. Default is empty ({}), so existing deployments are
    unaffected.

Motivation

Strimzi sizes the Kafka Connect JVM heap as a percentage of the cgroup memory
limit (STRIMZI_DYNAMIC_HEAP_PERCENTAGE, default 75). That works well in
production but is environment-sensitive: on some host kernel + container-runtime
combinations the cgroup-v2 read fails and the script falls back to host
/proc/meminfo, requesting a heap size larger than the container can hold and
SIGKILLing the JVM (OOMKilled, exit 137) shortly after startup.

Exposing jvmOptions lets affected environments (currently observed in local
kind-based dev stacks on newer kernels) pin -Xms/-Xmx explicitly without a
chart fork. The accompanying tiltronic PR documents the workaround for local dev.

Related PR: https://github.com/tidepool-org/tiltronic/pull/42

@clintonium-119 clintonium-119 changed the title fix(kafka): pin Kafka Connect heap via jvmOptions to avoid Strimzi dy… Pin Kafka Connect heap via jvmOptions to avoid Strimzi dynamic-heap fallback May 7, 2026
@clintonium-119 clintonium-119 deleted the fix-kafka-heap-jvm-options branch May 7, 2026 16:01
@clintonium-119 clintonium-119 restored the fix-kafka-heap-jvm-options branch May 7, 2026 16:06
@clintonium-119 clintonium-119 changed the title Pin Kafka Connect heap via jvmOptions to avoid Strimzi dynamic-heap fallback Expose Kafka Connect jvmOptions via chart values May 7, 2026
@clintonium-119 clintonium-119 reopened this May 7, 2026
Copy link
Copy Markdown
Contributor

@toddkazakov toddkazakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for the downstream effect. This is likely caused by the changes I did yesterday that add memory requests/limits. Without those strimzi doesn't set STRIMZI_DYNAMIC_HEAP_PERCENTAGE.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants