Commit Graph

19 Commits (1b7d5f31562cae4c37a07c5be7d9f2d52dcb7a69)

Author SHA1 Message Date
George 78cafa861b
refactor(notifications): isolate endpoint service (#19818)
* refactor(notifications): isolate endpoint service

Following the ongoing effort to isolate behaviours into their own
packages and off of kv.Service, this change move the notification
endpoints service implementation into its own package. It removes the
endpoint behaviors from the kv service completely.

* chore(influxd): wire up the isolated check service in place of kv service
2020-10-28 15:22:14 +00:00
Stuart Carnie 14c664daa4 chore: Format of url.Error message has changed
Provide a function to generate the error programmatically,
which is resilient to changes in Go versions.
2020-09-16 11:55:20 -07:00
Pavel Zavora 6d086f7495 feat(notification/telegram): take back telegram notification after being reverted in #19088
This reverts commit f67c3a14e0.
2020-08-12 11:28:02 +02:00
Michael Desa f67c3a14e0
Revert "feat(notification/telegram): add telegram notification" (#19088) 2020-07-27 08:31:18 -07:00
Pavel Zavora d85f89437e feat(notification/telegram): test error on empty endpoint name 2020-07-25 11:59:04 +02:00
Pavel Zavora 078bd8db09 feat(notification/telegram): specify channel as a telegram endpoint property 2020-07-25 11:59:04 +02:00
Pavel Zavora 192e997aea feat(notification/telegram): add telegram endpoint and rule 2020-07-25 11:59:04 +02:00
Jonathan A. Sternberg 0ae8bebd75
refactor: rewrite imports to include the /v2 suffix for version 2 2020-04-03 12:39:20 -05:00
Johnny Steenbergen 38606c66e1 feat(endpoint): drop id specific check for secret keys in all endpoints
the original design made the secrets unable to be reused, a bit to opinionated
to be useful eleswhere. This relaxes that requirement so that secrets can be
referenced here.
2019-12-17 17:10:27 -08:00
Johnny Steenbergen 86a359f34b chore(http): refactor http layer to be able to encode/decode influxdb.ID's safely
noticed that I had not used the http server as the entry point for server tests.
This was work to make that happen. Along the way, found a bunch of issues I hadn't
seen before 🤦. There are a number of changes tucked away inside the
other types, that make it possible to encode/decode a type with zero value for
influxdb.ID.
2019-12-13 09:46:01 -08:00
kelwang 3f168b109d
Merge pull request #15117 from influxdata/clientURL_optional
fix(notification/endpoint): pagerduty clientURL should be optional
2019-10-15 13:11:50 -04:00
Kelvin Wang aed8035472 fix(notification/endpoint): pagerduty clientURL should be optional 2019-09-11 15:26:50 -04:00
Jakub Bednar a8ad9e2615 docs(http): update NotificationEndpoint swagger definition to reflect reality 2019-09-11 11:53:44 +02:00
Michael Desa ea1b7bec09
fix(pagerduty/rule): change url to clientURL for pagerduty endpoint/rule
Closes https://github.com/influxdata/influxdb/issues/15042

Previously, there was an optional URL provided for the pagerduty
endpoint. However, the pagerduty API url does not change and as a result
it should not have been a parameter. The Pagerduty API does require a
`clientURL` that is presented in the pagerduty UI when an alert is
triggered. Currently that value will default to the alerts history page
for the organization.
2019-09-09 19:09:35 -04:00
Kelvin Wang d08122124f feat(notification/endpoint): add headers to http endpoint 2019-09-06 12:38:42 -04:00
Michael Desa 79a6b353e3
fix(notification/rule): conditionally include url and token for slack rule
Previously, we expected that both token and url would be provided, but
we've become aware that the user may provide either or both of them.
2019-09-05 18:00:45 -04:00
Kelvin Wang b933e007cb fix(notification/endpoint): remove slack token requirement 2019-09-05 16:48:34 -04:00
Michael Desa b26ed76d6a
fix(notification/check): ensure cloud integration works
fix(notification/check): include tags in check object in generated flux

Closes https://github.com/influxdata/influxdb/issues/14769

fix(notification/check): use selected field in threshold functions

Closes https://github.com/influxdata/influxdb/issues/14776

fix(testing): add selected field for check tests

fix(check): use real flux for threshold check

feat(notification/check): generate flux for deadman checks

chore(endpoint): rename webhook endpoint to http endpoint

fix(notification/rule): fetch url for flux script off of endpoint

fix(notification/rule): clean up slack and http rules

fix(notification/rule): change MessageTemp to MessageTemplate

fix(rules): pass endpoint in to rule during create

fix(ui): rename webhook to http

feat(notification/check): namespace deadman under alerts

fix(notification/check): nest tags under tags key in data object in flux

wip

feat(kv): log error if urm cannot be deleted for notification rule

fix(notification/rule): remove name from notify call in slack rule

chore(ui/cypress/e2e): skip rule create test
2019-08-27 15:02:53 -04:00
Kelvin Wang f683f0dc46 feat(notification/endpoint): add endpoint struct
Co-authored-by: Jade McGough <jade@thezets.com>
2019-08-15 21:56:17 -04:00