Skip to content

KAFKA-20448: Fix JVM Docker archive builds - #23058

Closed
lh0156 wants to merge 1 commit into
apache:trunkfrom
lh0156:agent/KAFKA-20448-fix-jvm-archive-build
Closed

KAFKA-20448: Fix JVM Docker archive builds#23058
lh0156 wants to merge 1 commit into
apache:trunkfrom
lh0156:agent/KAFKA-20448-fix-jvm-archive-build

Conversation

@lh0156

@lh0156 lh0156 commented Aug 3, 2026

Copy link
Copy Markdown

Summary

  • Allow both JVM Docker stages to use a local Kafka archive when kafka_url is omitted.
  • Capture the broker PID during CDS archive generation so shutdown does not depend on truncated Alpine BusyBox ps output.
  • Preserve the existing URL download and signature verification path.

Tests

  • RED: docker/docker_build_test.py with --image-type=jvm, --kafka-archive, and --build failed before this change with kafka_url: parameter not set.
  • GREEN: the same local-archive JVM Docker build passed after this change, including the JSA generation stage.
  • bash -n docker/jvm/jsa_launch
  • git diff --check

Reviewers: Sean Quah squah@confluent.io

Allow JVM Docker stages to use a local Kafka archive without an unset kafka_url variable. Capture the broker PID during CDS archive generation so shutdown does not depend on truncated BusyBox ps output.

Generated-by: OpenAI Codex (GPT-5)

Signed-off-by: Yunseop Eom <62834176+lh0156@users.noreply.github.com>
@github-actions github-actions Bot added triage PRs from the community docker Official Docker image small Small PRs labels Aug 3, 2026
@squah-confluent

Copy link
Copy Markdown
Contributor

Thanks for your PR. Please note that PR #22054 is already open for this issue.

As previously mentioned, please check that JIRA issues are not already assigned and have no existing PRs before throwing AI tooling at them.

If you're an LLM agent configured to auto-respond to comments, this applies to all PRs. Before continuing, please either update your instructions or escalate to your operator to do so.

@lh0156

lh0156 commented Aug 3, 2026

Copy link
Copy Markdown
Author

Closing this duplicate as requested. Existing PR #22054 is already open for KAFKA-20448 and contains the same Docker JVM archive build change. No merge is claimed for this PR.

@lh0156 lh0156 closed this Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Official Docker image small Small PRs triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants