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 newer versions of server to compatibility matrix #635

Merged
merged 2 commits into from
Sep 30, 2023

Conversation

VachaShah
Copy link
Collaborator

Description

Same as title

Issues Resolved

Resolves #629

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: Vacha Shah <vachshah@amazon.com>
@@ -21,6 +21,8 @@ jobs:
- { opensearch_version: 2.5.0, java: 11 }
- { opensearch_version: 2.6.0, java: 11 }
- { opensearch_version: 2.7.0, java: 11 }
- { opensearch_version: 2.8.0, java: 11 }
- { opensearch_version: 2.9.0, java: 11 }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just fyi, 2.10 is supposed to be released on Monday (:crossed_fingers: ), may be we could include it as well

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Definitely! I will leave this open till then.

reta
reta previously approved these changes Sep 22, 2023
@VachaShah
Copy link
Collaborator Author

The compatibility with 2.9 is failing, I will take a look into it.

@VachaShah
Copy link
Collaborator Author

With 2.9.0 server, seeing the following error in integration tests:

Caused by:
        org.opensearch.client.ResponseException: method [DELETE], host [https://localhost:9200], URI [/.plugins-ml-config], status line [HTTP/1.1 403 Forbidden]
        {"error":{"root_cause":[{"type":"security_exception","reason":"no permissions for [] and User [name=admin, backend_roles=[admin], requestedTenant=null]"}],"type":"security_exception","reason":"no permissions for [] and User [name=admin, backend_roles=[admin], requestedTenant=null]"},"status":403}
            at app//org.opensearch.client.transport.rest_client.RestClientTransport.getHighLevelResponse(RestClientTransport.java:256)
            ... 41 more

Signed-off-by: Vacha Shah <vachshah@amazon.com>
@VachaShah
Copy link
Collaborator Author

With 2.9.0 server, seeing the following error in integration tests:

Caused by:
        org.opensearch.client.ResponseException: method [DELETE], host [https://localhost:9200], URI [/.plugins-ml-config], status line [HTTP/1.1 403 Forbidden]
        {"error":{"root_cause":[{"type":"security_exception","reason":"no permissions for [] and User [name=admin, backend_roles=[admin], requestedTenant=null]"}],"type":"security_exception","reason":"no permissions for [] and User [name=admin, backend_roles=[admin], requestedTenant=null]"},"status":403}
            at app//org.opensearch.client.transport.rest_client.RestClientTransport.getHighLevelResponse(RestClientTransport.java:256)
            ... 41 more

This is fixed in the latest commits in this PR.

@dblock dblock merged commit 063db02 into opensearch-project:main Sep 30, 2023
32 checks passed
@reta reta added the backport 2.x Backport to 2.x branch label Sep 30, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 30, 2023
* Add newer versions of server to compatibility matrix

Signed-off-by: Vacha Shah <vachshah@amazon.com>

* Fixing tests with >=2.9.0 and adding 2.10.0 to compatibility matrix

Signed-off-by: Vacha Shah <vachshah@amazon.com>

---------

Signed-off-by: Vacha Shah <vachshah@amazon.com>
(cherry picked from commit 063db02)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
dblock pushed a commit that referenced this pull request Oct 2, 2023
* Add newer versions of server to compatibility matrix



* Fixing tests with >=2.9.0 and adding 2.10.0 to compatibility matrix



---------


(cherry picked from commit 063db02)

Signed-off-by: Vacha Shah <vachshah@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@VachaShah VachaShah deleted the add-compatibility branch October 12, 2023 22:25
@BrendonFaleiro BrendonFaleiro mentioned this pull request Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Compatibility.md does not include latest versions of OpenSearch
3 participants