Skip to content

[v13] Add Opsgenie plugin#28098

Merged
EdwardDowling merged 4 commits intobranch/v13from
backport-25995-branch/v13
Jun 23, 2023
Merged

[v13] Add Opsgenie plugin#28098
EdwardDowling merged 4 commits intobranch/v13from
backport-25995-branch/v13

Conversation

@EdwardDowling
Copy link
Copy Markdown
Contributor

@EdwardDowling EdwardDowling commented Jun 21, 2023

backport of #25995 and #23707

Need to backport these changes for https://github.com/gravitational/teleport.e/pull/1621 to work.

* Add initial opsgenie client

* Embed opsgenie client config in client and move some helper funcs

* Fix mod and sum and call to opsgenie client buildalertbody

* Add more godocs and use vendored teleport plugins lib

* Rename opsgenie client and config to prevent repetition

* Fix struct tag in opsgenie alertbody

* Fix misleading comment

* Add tests to opsgenie client to check note bodies are formatted

* Wrap errs with more descriptive types

* Close response bodies and fix error handling

* Add missing godoc and fix defer and return order

* Add missing godocs to opsgenie types

* Change opsgenie client test to not depend on json indent etc

* Add example of api url to godoc

* Undo unnecessary mod and sum changes

* Fix go sum

* Move opsgenie client code to integrations/lib

* Fix typos
@EdwardDowling EdwardDowling marked this pull request as ready for review June 21, 2023 13:46
@EdwardDowling EdwardDowling added this pull request to the merge queue Jun 23, 2023
@EdwardDowling EdwardDowling removed this pull request from the merge queue due to a manual request Jun 23, 2023
@EdwardDowling EdwardDowling added this pull request to the merge queue Jun 23, 2023
Merged via the queue into branch/v13 with commit 1aaa9b0 Jun 23, 2023
@EdwardDowling EdwardDowling deleted the backport-25995-branch/v13 branch June 23, 2023 14:43
@r0mant r0mant mentioned this pull request Jul 14, 2023
@fheinecke fheinecke mentioned this pull request Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants