Push a notification when a canary is waiting for approval

fluxcd · Jul 25, 2019 · 163f529 · 163f529
1 parent e07a82d
commit 163f529
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/docs/gitbook/how-it-works.md b/docs/gitbook/how-it-works.md
@@ -901,7 +901,8 @@ Manual gating with Flagger's tester:
         url: http://flagger-loadtester.test/gate/halt
 ```
 
-The `/gate/halt` returns HTTP 403 thus blocking the rollout.
+The `/gate/halt` returns HTTP 403 thus blocking the rollout. 
+If you have notifications enabled, Flagger will post a message to Slack or MS Teams if a canary rollout is waiting for approval.
 
 Change the URL to `/gate/approve` to start the canary analysis:
 

diff --git a/pkg/controller/scheduler.go b/pkg/controller/scheduler.go
@@ -537,11 +537,12 @@ func (c *Controller) runConfirmRolloutHooks(canary *flaggerv1.Canary) bool {
 			err := CallWebhook(canary.Name, canary.Namespace, flaggerv1.CanaryPhaseProgressing, webhook)
 			if err != nil {
 				if canary.Status.Phase != flaggerv1.CanaryPhaseWaiting {
-					c.recordEventWarningf(canary, "Halt %s.%s advancement waiting for approval %s",
-						canary.Name, canary.Namespace, webhook.Name)
 					if err := c.deployer.SetStatusPhase(canary, flaggerv1.CanaryPhaseWaiting); err != nil {
 						c.logger.With("canary", fmt.Sprintf("%s.%s", canary.Name, canary.Namespace)).Errorf("%v", err)
 					}
+					c.recordEventWarningf(canary, "Halt %s.%s advancement waiting for approval %s",
+						canary.Name, canary.Namespace, webhook.Name)
+					c.sendNotification(canary, "Canary is waiting for approval.", false, false)
 				}
 				return false
 			} else {