leaving channel unset silently breaks Slack provider about notification-controller HOT 2 CLOSED

I think all providers should have a validation func that could be called inside the controller reconciliation. If the validation fails, the ready status should be set to false with the validation error as the message.

from notification-controller.

So this can be considered either a documentation or validation issue.

The thing is that only some provider types require that channel to be set, and will fail if it is left empty. While other provider types do not.

Either way the errors needs to be made available in a more prominent location other than the logs. One option may be to expose the initialization errors as events in the provider resource.

from notification-controller.