-
Notifications
You must be signed in to change notification settings - Fork 42
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
LogPoller prune test #1351
base: max-logs-kept-tests
Are you sure you want to change the base?
LogPoller prune test #1351
Conversation
This was causing duplicate rows to get inserted during a replay instead of it being an UPDATE
@@ -658,7 +658,7 @@ func (lp *logPoller) backgroundWorkerRun() { | |||
} | |||
case <-logPruneTick: | |||
lp.lggr.Infof("Pruning LogPoller logs...") | |||
logPruneTick = time.After(utils.WithJitter(lp.pollPeriod * 2401)) // = 7^5 avoids common factors with 1000 | |||
logPruneTick = time.After(utils.WithJitter(5 * time.Minute)) // = 7^5 avoids common factors with 1000 |
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.
Ignoring poll period only to test
@@ -443,7 +443,8 @@ func NewCommitStore(lggr logger.Logger, addr common.Address, ec client.Client, l | |||
Name: logpoller.FilterName(v1_0_0.EXEC_REPORT_ACCEPTS, addr.String()), | |||
EventSigs: []common.Hash{eventSig}, | |||
Addresses: []common.Address{addr}, | |||
Retention: ccipdata.CommitExecLogsRetention, | |||
//Retention: ccipdata.CommitExecLogsRetention, | |||
MaxLogsKept: 10, |
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.
This is probably going to break the tests, because it's too aggressive. I set that intentionally to enforce more freq prunes
Quality Gate failedFailed conditions See analysis details on SonarQube Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
814b898
to
0a76672
Compare
e4aa477
to
6164cd5
Compare
bfc8e26
to
49648cc
Compare
Motivation
Solution