-
Notifications
You must be signed in to change notification settings - Fork 183
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
[FEATURE] Add Tests for Notifications Plugin #636
Comments
Yes, please contribute! I believe you can add the spec to https://github.com/opensearch-project/opensearch-api-specification and the client will auto-generate the methods. |
Thanks the the prompt reply, I raised opensearch-project/opensearch-api-specification#169 just to get this thing going, I'd appreciate your feedback there so that I can continue the work. Many thanks in advance! |
Notifications API Added: Updating Issue to Add Tests for the API. Please feel free to contribute. |
resolved |
Is your feature request related to a problem?
The plugins.alerting.AlertingClient.create_destination API does not work with latest OpenSearch clusters, as destinations have been deprecated in favor of Notification channels. As a result, using that API gives:
What solution would you like?
Add support for Notification APIs, I guess through a different plugin.
What alternatives have you considered?
One can still use transport.perform_request in order to use Notification channels APIs, but it would be nicer to have it natively supported on the python client.
The text was updated successfully, but these errors were encountered: