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.

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

Add JavaVersionChecker to check Java version for compatibility before… #13356

Merged
merged 3 commits into from
Oct 26, 2021

Conversation

robbavey
Copy link
Member

@robbavey robbavey commented Oct 25, 2021

… running

This commit provides a "friendly" error message when running Logstash with an incompatible version of Java, printing the following error message when attempting to start logstash:

"the minimum required Java version is 11; your Java version from [%s] does not meet this requirement",

This will prevent Logstash from running when at attempt is made to run using an older version of Logstash than java 11, which is the minimum requirement for Logstash 8.0

… running

Provides a "friendly" error message when running Logstash with an incompatible
version of Java
Improve readability of `JavaVersion`
Fix logstash bash script to exit on JavaVersionChecker error
@robbavey robbavey marked this pull request as ready for review October 26, 2021 19:38
@robbavey
Copy link
Member Author

Note:

This has been tested on MacOs and Windows

Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

LGTM, tested manually the multiple bin tools and it all works as expected 👍

@robbavey robbavey merged commit 620a654 into master Oct 26, 2021
robbavey added a commit to robbavey/logstash that referenced this pull request Oct 26, 2021
robbavey added a commit that referenced this pull request Oct 26, 2021
@robbavey robbavey deleted the check_java branch November 17, 2021 21:11
robbavey added a commit to robbavey/logstash that referenced this pull request Dec 16, 2021
… running

Provides a "friendly" error message when running Logstash with an incompatible
version of Java

Readded version of elastic#13356, incorporated with use of Comparable from elastic#13358
robbavey added a commit that referenced this pull request Jan 4, 2022
… running (#13533)

Provides a "friendly" error message when running Logstash with an incompatible
version of Java

Readded from #13356, incorporated with use of Comparable from #13358
robbavey added a commit to robbavey/logstash that referenced this pull request Jan 10, 2022
…n for compatibility before running

Backport PR#13433 to 8.0 branch. Original message:
Provides a "friendly" error message when running Logstash with an incompatible
version of Java

Readded from elastic#13356, incorporated with use of Comparable from elastic#13358
robbavey added a commit that referenced this pull request Jan 10, 2022
…n for compatibility before running (#13596)

Backport PR#13433 to 8.0 branch. Original message:
Provides a "friendly" error message when running Logstash with an incompatible
version of Java

Readded from #13356, incorporated with use of Comparable from #13358
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants