argoproj / notifications-engine

Affordable notifications for Kubernetes
Apache License 2.0
264 stars 134 forks source link

feat(opsgenie): Add support for setting priority in Opsgenie notificacation #266

Closed bofaalsarah closed 5 months ago

bofaalsarah commented 5 months ago

This PR adds support for setting the priority field in Opsgenie notifications. If a priority is provided in the notification, it will be included in the alert creation request to Opsgenie. Additionally, the code now checks for missing description in Opsgenie notifications and returns an error if it's missing.

codecov[bot] commented 5 months ago

Codecov Report

Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (c0913e2) 54.79% compared to head (52ac60d) 54.62%.

:exclamation: Current head 52ac60d differs from pull request most recent head 0f7aadd. Consider uploading reports for the commit 0f7aadd to get more accurate results

Files Patch % Lines
pkg/services/opsgenie.go 0.00% 12 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #266 +/- ## ========================================== - Coverage 54.79% 54.62% -0.18% ========================================== Files 35 35 Lines 3376 3387 +11 ========================================== Hits 1850 1850 - Misses 1254 1265 +11 Partials 272 272 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.