Skip to content

Commit

Permalink
fix: send workflow failure notifs to regular and failure channels
Browse files Browse the repository at this point in the history
  • Loading branch information
smrz2001 committed Nov 7, 2024
1 parent 4a845aa commit 539e51e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cd/manager/notifs/workflow.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,8 @@ func newWorkflowNotif(jobState job.JobState) (jobNotif, error) {

func (w workflowNotif) getChannels() []webhook.Client {
if w.state.Stage == job.JobStage_Failed {
return w.failureWebhooks
// Send to both the normal and failure webhooks
return append(w.webhooks, w.failureWebhooks...)
} else
// Skip "started" notifications so that the channel doesn't get too noisy
if w.state.Stage != job.JobStage_Started {
Expand Down

0 comments on commit 539e51e

Please sign in to comment.