-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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.
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
[Meta] [FEATURE] Add OPENSEARCH_JAVA_HOME env to override JAVA_HOME #1872
Comments
OpenSearch doesn't use the environment variable OPENSEARCH_JAVA_HOME as mentioned on https://opensearch.org/docs/latest/opensearch/install/compatibility/. Closing this ticket, since we already have another issue opened to fix the documentation. |
Should we turn this into a feature request? @chaitrahegde115 is there a reason why you'd want an ENV-set value vs. say replacing binaries in the distribution or consuming a no-JDK min version? |
To be clear, OpenSearch does support specifying a JDK with an environment variable, it's just that the environment variable is |
@dblock @jcgraybill that is correct, but it could be useful for certain cases (migration fe), Elasticsearch has this feature elastic/elasticsearch#68954. +1 to turn it into feature request, I actually do have an example from the field where it is useful. There are often several JVM colocated apps / services running on the same host which use different JDKs (fe apps go on edge with JDK-17 while services are still conservative on JDK-8/11). In this case, setting JAVA_HOME would not work (for all), but introducing OPENSEARCH_JAVA_HOME would actually make such deployments / migrations much simpler. |
I'll reopen as a feature request. |
@dblock if there are no objections, you could assign it to me, thx! |
@chaitrahegde115 original issue was as follows Describe the bug
But when I set JAVA_HOME to my java installation directory like below, it picked up the java version which I want to use.
Hence OPENSEARCH_JAVA_HOME doesn't seem to work. I turned it into a meta issue for adding support to |
@dblock could you please add this one as well opensearch-project/performance-analyzer-rca#127, thank you |
@dblock I think we are done with it? |
Coming from here
OPENSEARCH_JAVA_HOME
in this repo[FEATURE] Add OPENSEARCH_JAVA_HOME env to override JAVA_HOME #2001
[Backport] [1.x] [FEATURE] Add OPENSEARCH_JAVA_HOME env to override JAVA_HOME #2040
[Backport] [1.2] [FEATURE] Add OPENSEARCH_JAVA_HOME env to override JAVA_HOME (#2040) #2042
[BUG]
OPENSEARCH_JAVA_HOME
is incorrect documentation-website#364Related, opensearch-project/OpenSearch-Dashboards#1219
The text was updated successfully, but these errors were encountered: