Skip to content

Rescue `Redis::CannotConnectError` in `AuditEventStreamingWorker`

Michael Becker requested to merge fix/388026 into master

What does this MR do and why?

resolves #388026 (closed)

After removing the FF and rolling this feature out fully, we started seeing periodic failed jobs with a redis connection failure (example)

I think the only place this job is calling out to redis is to track metrics, and it seems more desirable to complete the job and log the failed metric tracking, than to fail the job entirely.

This might not free up ou error budget as we are still logging an exception, but at east the job will finish

How to set up and validate locally

CI passes

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Michael Becker

Merge request reports