-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[VRF-892] Remove noisy log poller warning for VRFv2(+) jobs #12132
Conversation
I see that you haven't updated any CHANGELOG files. Would it make sense to do so? |
} | ||
|
||
// on startup we want to initialize the last processed block | ||
if startingUp { | ||
var err2 error |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be fine naming it as err
instead of err2
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I think you're right that it wouldn't cause a shadowing issue.
Sorry for the re-review request here, the PR had an orphaned goroutine in the test so it was causing failures in GH actions. Also, added a git changeset. |
looks good but there is a merge conflict again. will re approve once that's resolved |
Quality Gate passedIssues Measures |
Removes logic that repeatedly calls the Log Poller's
RegisterFilter
function, which fires off a noisy warning for duplicates. Instead checks the log poller for an existing filter and registers one if not found.