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

Fixed alerting health check behavior when alerting cannot find its health task in Task Manager. #99564

Conversation

YulNaumenko
Copy link
Contributor

@YulNaumenko YulNaumenko commented May 7, 2021

Resolves #98796

Current PR provides the next changes in the logic for the retrieving alerting framework health:

  1. When the task for getting alerting health with "_id" : "task:Alerting-alerting_health_check", does not exists we are scheduling the new task for the health checks and returns the status by calling the alerting savedObjects internal repository.
  2. Added better errors handling and logging for the failing health checks.

Tested on the initial PR with reverting the change of the order here: https://github.com/elastic/kibana/pull/94968/files#diff-10bb6773064a1e6d04d4b1b5d5ad3954d8cfeb814b24d3c4db449c65528e1f0aR17 to reproduce the failing tests.

@YulNaumenko YulNaumenko self-assigned this May 7, 2021
@YulNaumenko YulNaumenko added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.14.0 v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels May 10, 2021
@YulNaumenko YulNaumenko marked this pull request as ready for review May 10, 2021 00:52
@YulNaumenko YulNaumenko requested a review from a team as a code owner May 10, 2021 00:52
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

References to deprecated APIs

id before after diff
canvas 29 25 -4
crossClusterReplication 8 6 -2
fleet 4 2 -2
globalSearch 4 2 -2
indexManagement 12 7 -5
infra 5 3 -2
licensing 18 15 -3
monitoring 109 56 -53
total -73

History

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

cc @YulNaumenko

@chrisronline chrisronline self-requested a review May 10, 2021 14:14
Copy link
Contributor

@chrisronline chrisronline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@YulNaumenko YulNaumenko merged commit fcc2ac5 into elastic:master May 10, 2021
YulNaumenko added a commit to YulNaumenko/kibana that referenced this pull request May 10, 2021
…alth task in Task Manager. (elastic#99564)

* Fixed alerting health check behavior when alerting cannot find its health task in Task Manager.

* fixed test

* added unit tests
YulNaumenko added a commit to YulNaumenko/kibana that referenced this pull request May 10, 2021
…alth task in Task Manager. (elastic#99564)

* Fixed alerting health check behavior when alerting cannot find its health task in Task Manager.

* fixed test

* added unit tests
YulNaumenko added a commit that referenced this pull request May 11, 2021
…alth task in Task Manager. (#99564) (#99728)

* Fixed alerting health check behavior when alerting cannot find its health task in Task Manager.

* fixed test

* added unit tests
YulNaumenko added a commit that referenced this pull request May 11, 2021
…alth task in Task Manager. (#99564) (#99727)

* Fixed alerting health check behavior when alerting cannot find its health task in Task Manager.

* fixed test

* added unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.14.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Status page returns 503 when alerting cannot find its health task in Task Manager
4 participants