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

[FEATURE] Support notifications #30

Closed
moritzzimmer opened this issue Mar 9, 2023 · 2 comments · Fixed by #68
Closed

[FEATURE] Support notifications #30

moritzzimmer opened this issue Mar 9, 2023 · 2 comments · Fixed by #68
Labels
enhancement New feature or request

Comments

@moritzzimmer
Copy link
Contributor

Is your feature request related to a problem?

Opensearch destinations seem to be deprecated with 2.0 and have been superseded by notifications, see https://opensearch.org/docs/latest/observing-your-data/alerting/monitors/#questions-about-destinations and https://docs.aws.amazon.com/opensearch-service/latest/developerguide/alerting.html#alerting-notifications.

Using a opensearch_destination ressource with this provider and a Opensearch domain 2.x fails with

│ Error: elastic: Error 405 (Method Not Allowed)

What solution would you like?

Support the notifications API in this provider: https://opensearch.org/docs/latest/observing-your-data/notifications/api/

What alternatives have you considered?

Create notifications using direct API calls or using the Dashboards UI.

Do you have any additional context?

no

@moritzzimmer moritzzimmer added enhancement New feature or request untriaged labels Mar 9, 2023
@gaiksaya
Copy link
Member

[Triage] Hi @opensearch-project/alerting-plugin , @opensearch-project/notifications Can you look into this issue?

@prudhvigodithi
Copy link
Collaborator

Hey @moritzzimmer thanks for the details, I believe at this point the provider is battle tested for 1.x versions, @phillbaker can you please add more details for 2.x?
Thanks you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants