Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Use ES_JAVA_HOME instead of JAVA_HOME #789

Merged
merged 1 commit into from
Mar 18, 2021
Merged

Use ES_JAVA_HOME instead of JAVA_HOME #789

merged 1 commit into from
Mar 18, 2021

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Mar 18, 2021

This commit set ES_JAVA_HOME environment variable instead of
JAVA_HOME when es_java_home environment variable is used.

See elastic/elasticsearch#68848 for more
context.

This commit set `ES_JAVA_HOME` environment variable instead of
`JAVA_HOME` when `es_java_home` environment variable is used.

See elastic/elasticsearch#68848 fro more
context.
@jmlrt jmlrt requested a review from a team March 18, 2021 09:40
Copy link
Contributor

@Conky5 Conky5 left a comment

Choose a reason for hiding this comment

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

LGTM

@jmlrt jmlrt merged commit ba1e828 into elastic:master Mar 18, 2021
@elasticmachine elasticmachine mentioned this pull request Mar 23, 2021
v1v added a commit to v1v/ansible-elasticsearch that referenced this pull request Apr 27, 2021
…buntu-20

* upstream/master:
  [meta] update ubuntu-1804 during kitchen provisioning (elastic#793)
  Stop plugin install to fail in check mode (elastic#787)
  7.12.0 release (elastic#790)
  Use ES_JAVA_HOME instead of JAVA_HOME (elastic#789)
  Fix deb and rpm url (elastic#788)
  [meta] fix changelog after 7.11.2 release (elastic#784)
  7.11.2 release (elastic#783)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants