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

Upgrade Elasticsearch client to 7.x #20248

Closed
wants to merge 1 commit into from

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Dec 30, 2023

Release notes:

Add support for Elasticsearch 8.x and drop support for Elasticsearch 6.x

This removes support for Elasticsearch 6.x while introducing support and test coverage for 8.x.

Both http security and user authentication are enabled by default.
@wendigo wendigo force-pushed the serafin/elasticsearch-client-7 branch from ebaa370 to e604886 Compare December 30, 2023 19:10
@mkhludnev
Copy link

It's rather cool, @wendigo. So, far it fails on Opensearch and Ngnix.

@wendigo
Copy link
Contributor Author

wendigo commented Jan 1, 2024

@mkhludnev yeah, i'm aware. Misconfiguration

@wendigo
Copy link
Contributor Author

wendigo commented Jan 1, 2024

Unfortunately that won't work. Despite ES 7 is compatible with ES 8.x, it breaks communication with opensearch as there as an artificial version checks that disallows communicating with OpenSearch 1.x and 2.x.

Any idea how to move forward @kokosing @findepi @martint ?

@wendigo wendigo closed this Jan 1, 2024
@mkhludnev
Copy link

mkhludnev commented Jan 1, 2024

Hold on, before get into this I encounter #11377, now it seems like it's mandatory. Will Opensearch connector be a separate plugin or we can hope both clients to reside in the same module? UPD: nope duplicate-finder ban ES&OS clients share a module.

@martint
Copy link
Member

martint commented Jan 1, 2024

See #11295 (comment)

@wendigo
Copy link
Contributor Author

wendigo commented Jan 2, 2024

OpenSearch support is moved to a separate connector via #20257

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants