GovukError.configure do |config|
  config.excluded_exceptions += [
    "AssetManagerAttachmentSetUploadedToWorker::AttachmentDataNotFoundTransient",
    "Redis::CannotConnectError",
  ]

  config.data_sync_excluded_exceptions += [
    "ActiveRecord::Deadlocked",
    "Mysql2::Error",
  ]
end

GovukError.configure do |config|
  config.excluded_exceptions += [
    "AssetManagerAttachmentSetUploadedToWorker::AttachmentDataNotFoundTransient",
    "Redis::CannotConnectError",
    "Sidekiq::JobRetry::Skip",
  ]

  config.data_sync_excluded_exceptions += [
    "ActiveRecord::Deadlocked",
    "Mysql2::Error",
  ]
end

Add Sidekiq::JobRetry::Skip to list of excluded exceptions
Add Sidekiq::JobRetry::Skip to list of excluded exceptions

By adding sentry-sidekiq in 596aba9c8d we were able to get Sidekiq
errors sent to Sentry. However we got these in addition to the
Sidekiq::JobRetry::Skip ones, this occurs because they aren't an
excluded exception. I thought adding this wouldn't be needed as the gem
does it automatically [1], however I believe our custom configuration
prevents this setting from taking effect.

I intend to put this exception into the list of govuk_app_config [2],
however I thought I'd put it directly in Whitehall first so could
establish if it's beneficial before creating a new gem release.

[1]: https://github.com/getsentry/sentry-ruby/blob/0fe8d27e4f84fda11475437a1cdd16f016efd904/sentry-sidekiq/lib/sentry/sidekiq/configuration.rb#L7
