You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We could remove the validation but in some cases the controller needs read access to the involved objects (Alert.spec.eventSources.matchLabels) and in some cases it needs write access (Receivers). To make the controller general-purpose, people would need to run it as cluster-admin so it can read/write to any involved objects in the cluster 😱
The main motivation would be to leverage existing notification controller capabilities to make it work beyond flux domain to any other business domain.
In our context, within weave gitops we have discussed to use in the scenario of pipelines, to notify / signal external systems around promotions, for example, a promotion is requested or promotion has happened
But it would be equally applicable to other domains like policy, to notify around policy domain events like policy has been created or a policy X has been violated for an application Y
We would like to use a notification controller as a general-purpose notification service to send non-flux domain events leveraging its capabilities.
We have found that the CRD spec validates against flux-domain types and an issue exists around this topic with the suggested solution of patching the crds.
This issue would be to request / start the conversation for an enhancement to notification controller that would not require patching flux-owned crds.
The text was updated successfully, but these errors were encountered: