Skip to content

fix(kafka): restore 40 percent automatic heap sizing#3248

Draft
weicao wants to merge 2 commits into
audrey/kafka-kraft-scalein-failclosedfrom
audrey/task55-heap-40
Draft

fix(kafka): restore 40 percent automatic heap sizing#3248
weicao wants to merge 2 commits into
audrey/kafka-kraft-scalein-failclosedfrom
audrey/task55-heap-40

Conversation

@weicao

@weicao weicao commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Problem

Task #54 formal runtime on addon candidate 67200df711766c01ec7a33829c1b0ee041b1f32a reached the regression R2 check with the live Kafka process using -XX:MaxRAMPercentage=100. The candidate templates and README contain the same 100 percent value, while merged PR #2855 is not in the candidate ancestry.

Change

Verification

  • bash -n passes for both server setup scripts.
  • Kafka ShellSpec: 30 examples, 0 failures on Bash 5.3.
  • Helm dependency build and helm template succeed.
  • Rendered chart and Kafka source/examples contain no MaxRAMPercentage=100.
  • Direct function check with a 1024 MiB limit produces -Xmx409m -Xms409m in both setup scripts.
  • git diff --check passes.

Runtime retest, package, runner, sideload, evidence, and cleanup remain tester-owned.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (67200df) to head (2067c10).

Additional details and impacted files
@@                          Coverage Diff                          @@
##           audrey/kafka-kraft-scalein-failclosed   #3248   +/-   ##
=====================================================================
  Coverage                                   0.00%   0.00%           
=====================================================================
  Files                                         76      76           
  Lines                                       9426    9426           
=====================================================================
  Misses                                      9426    9426           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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