Open ambis opened 3 years ago
Is this something that we should consider implementing? It will break existing multi tenant setups if we switch over to allowing cross namespace references.
👍
I personally would like the suggested behaviour. It would allow tenants to set up their own alerts using an existing integration.
Something like this would be useful for one of my clients: we have one provider for a lot of alerts and we want alerts to be created in same namespaces as Helm releases (and not as the provider) so each namespace would be more "self-contained".
I've opened a PR that solves this issue and a couple others :)
Running 0.6.1
Regarding this section: https://github.com/fluxcd/notification-controller/blob/main/config/crd/bases/notification.toolkit.fluxcd.io_alerts.yaml#L92
It only allows you to specify the name of the alert Provider, in my case discord. But it does not allow to reference it form another namespace (monitoring in this case).
The intent I believe, in a single tenant cluster, is to have only one Provider, which sits nicely in the
flux-system
namespace.But it is impossible now to create an alert in a namespace where there are eg. HelmReleases you want to watch (like monitoring).
This is what I must do right now:
This is what I would like to do:
For me, the best solution would be:
Have issue https://github.com/fluxcd/notification-controller/issues/71 implemented, since I could subscribe to all (single tenant) cluster events at once, without having to worry about setting up separate alerts per namespace.