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

feat(alertchannels): Add webhook alertchannel type #56

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

akosveres
Copy link
Collaborator

@akosveres akosveres commented Nov 20, 2024

  • Add webhook alertchannel type
  • Add general alertchannel configurations (send failure, degraded, recovery, ssl expiry)
  • Update docs, examples and tests

Fixes issue #31

* Add webhook alertchannel type
* Add general alertchannel configurations (send failure, degraded,
  recovery, ssl expiry)
* Update docs, examples and tests
| `opsgenie.apikey.fieldPath` | string; Key inside the secret or configmap | none |
| `webhook.name` | string; Name for the webhook | none |
| `webhook.url` | string; URL for the webhook | none |
| `webhook.webhookType` | string; TODO: can't determine what this is | none |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could someone please tell me what this is used for? Couldn't determine based on the docs or the console UI.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I looked into this and it seems like the value is not critical at all. It is mostly used as a bucket for monitoring and metrics purposes, and it may also be used in the frontend for display purposes. If the value is not set, or is set to an unsupported value, then it's effectively just the generic "WEBHOOK".

This is the list of valid values:

WEBHOOK_DISCORD
WEBHOOK_FIREHYDRANT
WEBHOOK_GITLAB_ALERT
WEBHOOK_SPIKESH
WEBHOOK_SPLUNK
WEBHOOK_MSTEAMS
WEBHOOK_TELEGRAM
WEBHOOK_CORALOGIX
WEBHOOK_ILERT

Note: It looks like you can provide an unsupported value as well, as I mentioned above. You will not get a validation error but it will behave the same as if no value was provided in the first place.

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

Successfully merging this pull request may close these issues.

2 participants