Additional logging and handling for missing schedule errors

- Hard to track error in integration where the ProcessPostcodeWorker can't affect the backoff service because it can't see the scheduler. Can't replicate on the console, so add a bit of logging and defaults so that Sentry isn't overwhelmed with this odd error.
alphagov · May 16, 2024 · f250fa6 · f250fa6
1 parent c2e5e2b
commit f250fa6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/db/schema.rb b/db/schema.rb
diff --git a/lib/sidekiq_scheduler_backoff_service.rb b/lib/sidekiq_scheduler_backoff_service.rb
@@ -24,10 +24,15 @@ def record_failure
   def current_interval
     schedule = Sidekiq.get_schedule[name]
     Integer(schedule["every"].chop)
+  rescue StandardError
+    Rails.logger.error("Queue schedule missing, available schedules: #{Sidekiq.get_schedule.keys.join(", ")}")
+    "1"
   end
 
   def restart_schedule(target_interval)
     schedule = Sidekiq.get_schedule[name]
     Sidekiq.set_schedule(name, schedule.merge("every" => "#{target_interval}s"))
+  rescue StandardError
+    Rails.logger.error("Queue schedule missing, available schedules: #{Sidekiq.get_schedule.keys.join(", ")}")
   end
 end