Skip to content

Added support for version >= 8.0.0 #46

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

Merged
merged 5 commits into from
Jun 30, 2023
Merged

Conversation

brendon1982
Copy link
Contributor

@brendon1982 brendon1982 commented Aug 22, 2022

The issue with using version 8 was that elasticsearch-certutil no longer generated a CA automatically. It is now a separate step in the setup. The GenerateCertificatesTask now generates a certificate authority when version 8 detected.

In addition version 8 also produces a slightly different warning message when JAVA_HOME is missing. The ClusterComposeTask ExecuteBinary now also ignores warning lines containing "using ES_JAVA_HOME".

* GenerateCertificatesTask now generates a certificate authority when version 8 detected
* ClusterComposeTask ExecuteBinary now also ignores lines containing "using ES_JAVA_HOME"
* Added check back in GenerateCertificatesTask to skip certificate generation if certificate path already exists
@cla-checker-service
Copy link

cla-checker-service bot commented Aug 22, 2022

💚 CLA has been signed

@Mpdreamz
Copy link
Member

Wow this is great and completely flew under my radar, apologies for getting to it so late.

Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

LGTM! 👍 💯

@Mpdreamz Mpdreamz merged commit df50252 into elastic:master Jun 30, 2023
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.

2 participants