From 425cdf3f509a14bbee4cd56cb1dae1635d99024f Mon Sep 17 00:00:00 2001 From: Lukas Erlacher Date: Mon, 19 Jan 2026 10:02:03 +1100 Subject: [PATCH] PagerDuty: Remove oncall check from maint start maint start checking for oncall status just creates annoyance during an incident response. There's no reason to restrict it - there's no potential for misuse, and we have the periodic reminder as well. --- lib/Synergy/Reactor/PagerDuty.pm | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/lib/Synergy/Reactor/PagerDuty.pm b/lib/Synergy/Reactor/PagerDuty.pm index 2fef283f..a7a3f9aa 100644 --- a/lib/Synergy/Reactor/PagerDuty.pm +++ b/lib/Synergy/Reactor/PagerDuty.pm @@ -235,19 +235,6 @@ responder 'maint-start' => { return await $event->error_reply("I don't know you, so I'm ignoring that."); } - unless ($force) { - my $is_oncall = await $self->_user_is_oncall($event->from_user); - - unless ($is_oncall) { - return await $event->error_reply(join(q{ }, - "You don't seem to be on call right now.", - "Usually, the person oncall is getting the alerts, so they should be", - "the one to decide whether or not to shut them up.", - "If you really want to do this, try again with /force." - )); - } - } - my @maints = await $self->_relevant_maint_windows; if (@maints) {