docs: clarify the service in notificatioin.md (argoproj#1546)

- template and trigger should be included in the argo-rollouts-notification-configmap
- add debug message for missing notification message

Signed-off-by: Hui Kang <[email protected]>

Co-authored-by: Hui Kang <[email protected]>

Author: huikang

docs/features/notifications.md

@@ -35,6 +35,8 @@ kind: ConfigMap
 metadata:
   name: argo-rollouts-notification-configmap
 data:
+  # detail of the templates is omitted
+  # detail of the triggers is omitted
   service.slack: |
     token: $slack-token
 ---
@@ -124,7 +126,7 @@ when rollout pod spec uses `argoproj/rollouts-demo:purple` image:
 apiVersion: v1
 kind: ConfigMap
 metadata:
-  name: argocd-notifications-cm
+  name: argo-rollouts-notification-configmap
 data:
   trigger.on-purple: |
     - send: [my-purple-template]

utils/record/record.go

@@ -167,13 +167,15 @@ func NewAPIFactorySettings() api.Settings {
 
 // Send notifications for triggered event if user is subscribed
 func (e *EventRecorderAdapter) sendNotifications(object runtime.Object, opts EventOptions) error {
+	logCtx := logutil.WithObject(object)
 	subsFromAnnotations := subscriptions.Annotations(object.(metav1.Object).GetAnnotations())
 	destByTrigger := subsFromAnnotations.GetDestinations(nil, map[string][]string{})
 
 	trigger := translateReasonToTrigger(opts.EventReason)
 
 	destinations := destByTrigger[trigger]
 	if len(destinations) == 0 {
+		logCtx.Debugf("No configured destinations for trigger: %s", trigger)
 		return nil
 	}
 
@@ -185,6 +187,7 @@ func (e *EventRecorderAdapter) sendNotifications(object runtime.Object, opts Eve
 	// Creates config for notifications for built-in triggers
 	triggerActions, ok := notificationsAPI.GetConfig().Triggers[trigger]
 	if !ok {
+		logCtx.Debugf("No configured template for trigger: %s", trigger)
 		return nil
 	}