Skip to content
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

Specify Error Retry Wait and Error Limit for All Detection Engines #12957

Merged
merged 1 commit into from
May 7, 2024

Conversation

coreyogburn
Copy link
Contributor

If a sync errors out, the engine will wait communityRulesImportErrorSeconds seconds instead of the usual communityRulesImportFrequencySeconds seconds wait.

If failAfterConsecutiveErrorCount errors happen in a row when syncing detections to ElasticSearch then the sync is considered a failure and will give up and try again later. This assumes ElasticSearch is the source of the errors and backs of in hopes it'll be able to fix itself.

If a sync errors out, the engine will wait `communityRulesImportErrorSeconds` seconds instead of the usual `communityRulesImportFrequencySeconds` seconds wait.

If `failAfterConsecutiveErrorCount` errors happen in a row when syncing detections to ElasticSearch then the sync is considered a failure and will give up and try again later. This assumes ElasticSearch is the source of the errors and backs of in hopes it'll be able to fix itself.
@coreyogburn coreyogburn merged commit cb7dea1 into 2.4/dev May 7, 2024
2 checks passed
@coreyogburn coreyogburn deleted the cogburn/retry-import branch May 7, 2024 17:20
@github-actions github-actions bot locked and limited conversation to collaborators May 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants