Skip to content

[7.x] Implemented Alerting health status pusher by using task manager and status pooler for Kibana status plugins 'kibanahost/api/status' (#79056)#82907

Merged
YulNaumenko merged 1 commit intoelastic:7.xfrom
YulNaumenko:backport/7.x/pr-79056
Nov 7, 2020

Conversation

@YulNaumenko
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…tatus pooler for Kibana status plugins 'kibanahost/api/status' (elastic#79056)

* Implemented Alerting health status pusher by using task manager and status pooler for Kibana status plugins 'kibanahost/api/status'

* Exposed health task registration to alerts plugin

* Fixed type error

* Extended health API endpoint with info about decryption failures, added correct health task implementation

* adjusted query

* Tested locally and got it working as expected, fixed tests and type check

* Added unit tests

* Changed AlertExecutionStatusErrorReasons to be enum

* Uppercase the enum

* Replaced string values to enum

* Fixed types

* Extended AlertsClient with getHealth method

* added return type to healthStatus$

* Added configurable health check interval and timestamps

* Extended update core status interval to 5mins

* Fixed failing tests

* Registered alerts config

* Fixed date for ok health state

* fixed jest test

* fixed task state

* Fixed due to comments, moved getHealth to a plugin level

* fixed type checks

* Added sorting to the latest Ok state last update

* adjusted error queries

* Fixed jest tests

* removed unused

* fixed type check
@YulNaumenko YulNaumenko requested review from a team as code owners November 7, 2020 00:22
@YulNaumenko YulNaumenko added the backport This PR is a backport of another PR label Nov 7, 2020
@YulNaumenko YulNaumenko removed request for a team November 7, 2020 00:23
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

distributable file count

id before after diff
default 43107 43112 +5

page load bundle size

id before after diff
alerts 87.8KB 88.4KB +665.0B

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

@YulNaumenko YulNaumenko merged commit c2d04d3 into elastic:7.x Nov 7, 2020
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.

2 participants