Skip to content

Fix JAVA_HOME in Dockerimage and allow it to be overwritten#2449

Merged
Ghatage merged 1 commit into
apache:masterfrom
eolivelli:fix/docker-image-jdk11-2
Oct 21, 2020
Merged

Fix JAVA_HOME in Dockerimage and allow it to be overwritten#2449
Ghatage merged 1 commit into
apache:masterfrom
eolivelli:fix/docker-image-jdk11-2

Conversation

@eolivelli

Copy link
Copy Markdown
Contributor
  • Fix Java 11 Path in Dockerfile
  • Allow to overwrite JAVA_HOME in entrypoint.sh and healthcheck.sh

@eolivelli eolivelli marked this pull request as ready for review October 20, 2020 10:22
@eolivelli

Copy link
Copy Markdown
Contributor Author

@ravisharda @jiazhai PTAL

@eolivelli eolivelli requested review from Ghatage and jiazhai October 20, 2020 12:59

@ravisharda ravisharda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@Ghatage Ghatage added this to the 4.12.0 milestone Oct 21, 2020
@Ghatage Ghatage merged commit d806c22 into apache:master Oct 21, 2020
@Ghatage

Ghatage commented Oct 21, 2020

Copy link
Copy Markdown
Contributor

LGTM, I tested this out locally, forgot to approve.
Merged none-the-less.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants