Skip to content

Conversation

@payert
Copy link
Contributor

@payert payert commented Dec 12, 2025

KNOX-3222 - Integrate OpenSearch REST API in Knox

What changes were proposed in this pull request?

The change adds OpenSearch REST API support to knox.

How was this patch tested?

  • Manual testing: cluster with Knox and OpenSearch has been deployed where the discovery of OpenSearch services has been tested. Furthermore, the the proxying of API has been tested with PAM and JWT based authentications.
  • Unit tests has been executed.
  • System test suite has been executed.

Integration Tests

Unit tests has been added to cover the newly introduced service discovery code.

UI changes

No UI changes involved.

@github-actions
Copy link

Test Results

7 tests   7 ✅  1s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 5577485.

Copy link
Contributor

@smolnar82 smolnar82 left a comment

Choose a reason for hiding this comment

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

LGTM

@smolnar82 smolnar82 merged commit d99996a into apache:master Dec 12, 2025
3 checks passed
@smolnar82
Copy link
Contributor

Thanks, @payert, for your contribution!

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.

2 participants