Skip to content

[7.x] [Monitoring] Out of the box alerting (#68805)#71780

Merged
chrisronline merged 1 commit intoelastic:7.xfrom
chrisronline:backport/7.x/pr-68805
Jul 15, 2020
Merged

[7.x] [Monitoring] Out of the box alerting (#68805)#71780
chrisronline merged 1 commit intoelastic:7.xfrom
chrisronline:backport/7.x/pr-68805

Conversation

@chrisronline
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* First draft, not quite working but a good start

* More working

* Support configuring throttle

* Get the other alerts working too

* More

* Separate into individual files

* Menu support as well as better integration in existing UIs

* Red borders!

* New overview style, and renamed alert

* more visual updates

* Update cpu usage and improve settings configuration in UI

* Convert cluster health and license expiration alert to use legacy data model

* Remove most of the custom UI and use the flyout

* Add the actual alerts

* Remove more code

* Fix formatting

* Fix up some errors

* Remove unnecessary code

* Updates

* add more links here

* Fix up linkage

* Added nodes changed alert

* Most of the version mismatch working

* Add kibana mismatch

* UI tweaks

* Add timestamp

* Support actions in the enable api

* Move this around

* Better support for changing legacy alerts

* Add missing files

* Update alerts

* Enable alerts whenever any page is visited in SM

* Tweaks

* Use more practical default

* Remove the buggy renderer and ensure setup mode can show all alerts

* Updates

* Remove unnecessary code

* Remove some dead code

* Cleanup

* Fix snapshot

* Fixes

* Fixes

* Fix test

* Add alerts to kibana and logstash listing pages

* Fix test

* Add disable/mute options

* Tweaks

* Fix linting

* Fix i18n

* Adding a couple tests

* Fix localization

* Use http

* Ensure we properly handle when an alert is resolved

* Fix tests

* Hide legacy alerts if not the right license

* Design tweaks

* Fix tests

* PR feedback

* Moar tests

* Fix i18n

* Ensure we have a control over the messaging

* Fix translations

* Tweaks

* More localization

* Copy changes

* Type
# Conflicts:
#	x-pack/plugins/monitoring/common/constants.ts
#	x-pack/plugins/monitoring/public/components/cluster/overview/alerts_panel.js
#	x-pack/plugins/monitoring/public/components/cluster/overview/index.js
#	x-pack/plugins/monitoring/public/components/elasticsearch/node/node.js
#	x-pack/plugins/monitoring/public/components/elasticsearch/nodes/nodes.js
#	x-pack/plugins/monitoring/public/components/kibana/instances/instances.js
#	x-pack/plugins/monitoring/server/plugin.ts
#	x-pack/test/functional/apps/monitoring/cluster/alerts.js
@chrisronline chrisronline added the backport This PR is a backport of another PR label Jul 14, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
monitoring 217 +15 202

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

@chrisronline chrisronline merged commit 510a684 into elastic:7.x Jul 15, 2020
@chrisronline chrisronline deleted the backport/7.x/pr-68805 branch July 15, 2020 00:13
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