Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Usage of JAVA_HOME is deprecated in favor of ES_JAVA_HOME #68848

Closed
mark-vieira opened this issue Feb 10, 2021 · 1 comment · Fixed by #68954
Closed

Usage of JAVA_HOME is deprecated in favor of ES_JAVA_HOME #68848

mark-vieira opened this issue Feb 10, 2021 · 1 comment · Fixed by #68954
Assignees
@jasontedor
Labels
:Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts Team:Delivery Meta label for Delivery team

Comments

@mark-vieira
Copy link
Contributor

This relates to #55820 which intends to remove the usage of JAVA_HOME environment variable entirely for the purposes of locating a JRE for running Elasticsearch. For 7.x we want to first do the following:

  1. Add ES_JAVA_HOME as an option for supplying the JRE used for Elasticsearch.
  2. ES_JAVA_HOME will take precedence over JAVA_HOME.
  3. Emit deprecation message if JAVA_HOME is set but ES_JAVA_HOME isn't.
@mark-vieira mark-vieira added the :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts label Feb 10, 2021
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Feb 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@mark-vieira mark-vieira mentioned this issue Feb 10, 2021
Closed
@ghost ghost mentioned this issue Feb 11, 2021
Closed
4 tasks
@jasontedor jasontedor mentioned this issue Feb 17, 2021
Merged
@jmlrt jmlrt mentioned this issue Mar 18, 2021
Merged
jmlrt added a commit to jmlrt/ansible-elasticsearch that referenced this issue Mar 18, 2021
@jmlrt
Use ES_JAVA_HOME instead of JAVA_HOME
f6deb75 
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 added a commit to elastic/ansible-elasticsearch that referenced this issue Mar 18, 2021
@jmlrt
Use ES_JAVA_HOME instead of JAVA_HOME (#789)
ba1e828 
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.
smartech7 pushed a commit to smartech7/ruby-ansible-elasticsearch that referenced this issue Aug 4, 2023
@jmlrt
Use ES_JAVA_HOME instead of JAVA_HOME (#789)
56afc1b 
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasontedor jasontedor

Labels
:Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts Team:Delivery Meta label for Delivery team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Introduce ES_JAVA_HOME jasontedor/elasticsearch
3 participants
@mark-vieira @jasontedor @elasticmachine