If applied, this commit will add optional canary spec fields, summary…

… and cluster, for notification purposes

Signed-off-by: baldey-nz <baldey@gmail.com>

artifacts/flagger/crd.yaml

@@ -79,6 +79,9 @@ spec:
                 - service
                 - analysis
               properties:
+                summary:
+                  description: Optional msg to add to each notification
+                  type: string
                 provider:
                   description: Traffic managent provider
                   type: string
@@ -103,6 +106,9 @@ spec:
                         - Service
                     name:
                       type: string
+                    cluster:
+                      description: Optional msg to add to Target notification line
+                      type: string
                 autoscalerRef:
                   description: HPA selector
                   type: object

pkg/apis/flagger/v1beta1/canary.go

@@ -103,6 +103,10 @@ type CanarySpec struct {
 	// revert canary mutation on deletion of canary resource
 	// +optional
 	RevertOnDeletion bool `json:"revertOnDeletion,omitempty"`
+
+	// Summary creates new notification message
+	// +optional
+	Summary string `json:"summary,omitempty"`
 }
 
 // CanaryService defines how ClusterIP services, service mesh or ingress routing objects are generated
@@ -377,6 +381,10 @@ type CrossNamespaceObjectReference struct {
 	// Namespace of the referent
 	// +optional
 	Namespace string `json:"namespace,omitempty"`
+
+	// Cluster name will append to Target notification msg
+	// +optional
+	Cluster string `json:"cluster,omitempty"`
 }
 
 // CustomMetadata holds labels and annotations to set on generated objects.

pkg/controller/events.go

@@ -175,10 +175,20 @@ func (c *Controller) alert(canary *flaggerv1.Canary, message string, metadata bo
 
 func alertMetadata(canary *flaggerv1.Canary) []notifier.Field {
 	var fields []notifier.Field
+
+	if canary.Spec.Summary != "" {
+		fields = append(fields,
+			notifier.Field{
+				Name:  "Summary",
+				Value: canary.Spec.Summary,
+			},
+		)
+	}
+
 	fields = append(fields,
 		notifier.Field{
 			Name:  "Target",
-			Value: fmt.Sprintf("%s/%s.%s", canary.Spec.TargetRef.Kind, canary.Spec.TargetRef.Name, canary.Namespace),
+			Value: fmt.Sprintf("%s/%s.%s %s", canary.Spec.TargetRef.Kind, canary.Spec.TargetRef.Name, canary.Namespace, canary.Spec.TargetRef.Cluster),
 		},
 		notifier.Field{
 			Name:  "Failed checks threshold",