Skip to content

test: ensure java is on PATH in integration container#2066

Closed
Technoboy- wants to merge 1 commit into
codex/base-394981dfrom
codex/jdk-only-394981d
Closed

test: ensure java is on PATH in integration container#2066
Technoboy- wants to merge 1 commit into
codex/base-394981dfrom
codex/jdk-only-394981d

Conversation

@Technoboy-
Copy link
Copy Markdown
Contributor

Summary

  • add a small PATH/JAVA_HOME fallback in the integration test entrypoint
  • look for java in the JDK locations used by the 4.3.0-SNAPSHOT snstage image
  • keep this PR scoped to the JDK fix only for validation

Testing

  • validated that /opt/jvm/bin/java exists in snstage/pulsar-all:4.3.0-SNAPSHOT
  • validated the entrypoint script with bash -n
  • observed sink/source integration failures move past the previous "java not found" error after this fix

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

@Technoboy-:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions Bot added the doc-info-missing This pr needs to mark a document option in description label May 1, 2026
@Technoboy- Technoboy- closed this May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-info-missing This pr needs to mark a document option in description

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant