robusta-dev / kubewatch

Watch k8s events and trigger Handlers
Apache License 2.0
547 stars 79 forks source link

Helm: add extraHandlers section in configuration #29

Closed bmalynovytch closed 1 year ago

bmalynovytch commented 1 year ago

Fixes #28

bmalynovytch commented 1 year ago

I personally think that maintaining a static list of handlers isn't a good idea because the chart will require to be updated if new handlers come in. There's currently no check done on the values per handler, which means it's just overhead IMHO. If we add an entry for slackWebhook, then we'll probably also need to add an extraHandlers section to let people handle edge cases.