From 190c2b5122e81b478558abc8024b3a08813f9de0 Mon Sep 17 00:00:00 2001 From: Ben Thorner Date: Tue, 5 Apr 2022 16:15:02 +0100 Subject: [PATCH] Fix broadcast alert not triggering PagerDuty We previously decided against this [^1] but the lambda alerts are not a replacement for this one. Even if both sets of alerts go off, we expect PagerDuty to aggregate them into a single notification. [^1]: https://github.com/alphagov/notifications-api/commit/65c21f694c98ea838abab6dd6d01c4cb07aa8ca3 --- app/broadcast_message/utils.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/broadcast_message/utils.py b/app/broadcast_message/utils.py index e708e31a2..0aa500eca 100644 --- a/app/broadcast_message/utils.py +++ b/app/broadcast_message/utils.py @@ -90,7 +90,8 @@ def _create_p1_zendesk_alert(broadcast_message): technical_ticket=True, org_id=current_app.config['BROADCAST_ORGANISATION_ID'], org_type='central', - service_id=str(broadcast_message.service_id) + service_id=str(broadcast_message.service_id), + p1=True ) zendesk_client.send_ticket_to_zendesk(ticket)