OplogToRedis: Better Logging for Denylist Failures

[alex-goodisman]

Summary

When the denylist fails to update due to a database error, the error is now logged.

Jira Task

• No ticket needed (check only if level of impact is No-op)

Level of Impact

Please select exactly one option. To change the level of impact, first unselect the current level of impact and then select the new level of impact.

• 1 - High
• 2 - Medium
• 3 - Low
• 4 - No-op

Impact Analysis

This is a logging only change, and it's not an error level log.

Release Notes

none

Release Change Category

Please select exactly one option. To change the release change category, first unselect the current category and then select the new one.

• 1 - Internal Only (Non Customer Facing)
• 2 - Bug Fix (Customer Facing)
• 3 - New Feature
• 4 - Feature Enhancement
• 5 - Feature Deprecated
• 6 - Release Change Category Not Needed

Feature Flags

none

Test Plan

n/a

Checklist

Check each when they are done or confirmed to not apply. If something here isn't checked, this isn't ready to be merged. Seriously. You are making your colleagues lives a lot easier by doing this consistently. Be a good Tulipian.

• Test Plan Fully Executed
• Cypress Tests
• Does not require security review or security review complete

For more information on the requirements for every pull request, see Development Definition of Done.

[eparker-tulip]

Looks good -- logging proved to show what was needed. Nothing more should be needed here in OTR apart from bumping the minor version in the default.nix file (make it 3.8.5 since I have an open PR with 3.8.4 already)