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

Added new OpenSearch versions and updated compatibility matrix #257

Merged

Conversation

saimedhi
Copy link
Collaborator

Signed-off-by: saimedhi saimedhi@amazon.com

Description

Added new OpenSearch versions and updated compatibility matrix

Issues Resolved

Closes #252

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: saimedhi <saimedhi@amazon.com>
@wbeckler
Copy link
Contributor

Do you know if client v2.0 is compatible with older versions of the server, such as 1.3?

dblock
dblock previously approved these changes Jan 2, 2023
Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

I'm ok without a better changelog, but I think users would appreciate it.

CHANGELOG.md Outdated
@@ -9,6 +9,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
- Added overload decorators to helpers-actions.pyi-"bulk" ([#239](https://github.com/opensearch-project/opensearch-py/pull/239))
- Document Keberos authenticaion ([214](https://github.com/opensearch-project/opensearch-py/pull/214))
- Add release workflows ([#240](https://github.com/opensearch-project/opensearch-py/pull/240))
- Added new OpenSearch versions and updated compatibility matrix ([#257](https://github.com/opensearch-project/opensearch-py/pull/257))
Copy link
Member

Choose a reason for hiding this comment

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

We should be explicit here and say something like "Compatibility with OpenSearch 2.1.0 - 2.4.1".

@dblock
Copy link
Member

dblock commented Jan 2, 2023

Do you know if client v2.0 is compatible with older versions of the server, such as 1.3?

It should be according to https://github.com/opensearch-project/opensearch-py/blob/main/.github/workflows/integration.yml#L14.

@wbeckler
Copy link
Contributor

wbeckler commented Jan 2, 2023

@saimedhi I would suggest adding in the older versions to the matrix that are compatible, as the presumption for users would be that the lack of a version means there exists an incompatibility.

@saimedhi saimedhi force-pushed the fix/compatibility_matrix branch from f091590 to 06d9334 Compare January 5, 2023 06:18
harshavamsi
harshavamsi previously approved these changes Jan 5, 2023
Copy link
Collaborator

@harshavamsi harshavamsi left a comment

Choose a reason for hiding this comment

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

LGTM!

COMPATIBILITY.md Outdated

| Client Version | OpenSearch Version | Notes |
| --- | --- | --- |
| 1.0.0 | 1.0.0-1.0.1, 1.2.1-1.2.4 | |
Copy link
Collaborator

Choose a reason for hiding this comment

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

It might be a typo in the previous markdown, 1.1.0 client version was released with being compatible from OpenSearch versions 1.3.0. Can you please update that?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@VachaShah Made the changes as suggested.

@saimedhi saimedhi force-pushed the fix/compatibility_matrix branch 2 times, most recently from 12d4582 to bd888e4 Compare January 5, 2023 20:23
Signed-off-by: saimedhi <saimedhi@amazon.com>
@saimedhi saimedhi force-pushed the fix/compatibility_matrix branch from bd888e4 to 7faa02f Compare January 5, 2023 20:28
@saimedhi saimedhi requested review from VachaShah and harshavamsi and removed request for VachaShah January 5, 2023 20:34
@harshavamsi harshavamsi merged commit b4bfde0 into opensearch-project:main Jan 5, 2023
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.

[BUG] Client/Core Engine Compatibility Matrix Wrong/Outdated
5 participants