Skip to content

[8.8] [Synthetics] adjust alert timing (#159511)#159515

Merged
kibanamachine merged 1 commit intoelastic:8.8from
kibanamachine:backport/8.8/pr-159511
Jun 12, 2023
Merged

[8.8] [Synthetics] adjust alert timing (#159511)#159515
kibanamachine merged 1 commit intoelastic:8.8from
kibanamachine:backport/8.8/pr-159511

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.8:

Questions ?

Please refer to the Backport tool documentation

### Release note

Resolves elastic#158172

Fixes an issue where alerting on Synthetics monitors could sometimes
become delayed.

## Summary
Adjust alerting timing to account for alerting framework delays.

Alert executors do not fire perfectly on the specified interval (in our
case, every 1 minute). They are typically delayed a few seconds, be can
especially delayed if the customer does not have their alerting settings
configured to scale and there is no CPU available to take on the
scheduled task.

This PR accounts for small delays in the alerting framework, by
subtracting an additional 1 minute from the `from` value for our current
status query.

(cherry picked from commit d549500)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 12, 2023
@kibanamachine kibanamachine enabled auto-merge (squash) June 12, 2023 19:15
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 17 19 +2
securitySolution 400 404 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 18 20 +2
securitySolution 480 484 +4
total +6

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @dominiqueclarke

@kibanamachine kibanamachine merged commit d252e30 into elastic:8.8 Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants