-
Notifications
You must be signed in to change notification settings - Fork 923
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
[Multiple Datasource] Support serverless AWS OpenSearch in SigV4 auth type #3623
Comments
To support serverless for legacy datasource client is not as straight-forward as new client, because unlike new clients, legacy client package
For 1), we solved it in MD while releasing the sigV4 support in 2.6, by using a third party Connection handler for Amazon ES, https://github.com/TheDeveloper/http-aws-es. OpenSearch-Dashboards/src/plugins/data_source/server/legacy/configure_legacy_client.ts Line 10 in 9d27023
OpenSearch-Dashboards/src/plugins/data_source/server/legacy/configure_legacy_client.ts Lines 197 to 208 in 9d27023
For 2), since the Proposal
|
this project will be @shanilpa for UX/UI support, @zhongnansu |
Closing per @zhongnansu |
Problem statement
Multiple datasource should be able to connect to serverless AWS OpenSearch: https://aws.amazon.com/opensearch-service/features/serverless/
UI/UX changes (proposal only)
To create a serverless data source, there needs to be a way for user to specify serverless opensearch. I propose a new required field
service type
in the "create data source" flow, with optionsAWS OpenSearch
andAWS OpenSearch serverless
. Below image only explains the proposal. Please help with actual UI mock-up @shanilpa, and the correct naming.Task breakdown
test_connection
to remove the use of.info()
API, because.info()
API is not supported in serverless. Useping()
insteadservice
, candidates values arees
oraoss
Reference:
Opensearch-js client 2.x has support for serverless Opensearch: [FEATURE] Add support for OpenSearch Serverless opensearch-js#356
Similar issue: SigV4 supports different service name and other service's test connection #3718
The text was updated successfully, but these errors were encountered: