Skip to content

[7.x] [Monitoring] Missing data alert (#78208)#79163

Merged
chrisronline merged 1 commit intoelastic:7.xfrom
chrisronline:backport/7.x/pr-78208
Oct 1, 2020
Merged

[7.x] [Monitoring] Missing data alert (#78208)#79163
chrisronline merged 1 commit intoelastic:7.xfrom
chrisronline:backport/7.x/pr-78208

Conversation

@chrisronline
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* WIP for alert

* Surface alert most places

* Fix up alert placement

* Fix tests

* Type fix

* Update copy

* Add alert presence to APM in the UI

* Fetch data a little differently

* We don't need moment

* Add tests

* PR feedback

* Update copy

* Fix up bug around grabbing old data

* PR feedback

* PR feedback

* Fix tests
# Conflicts:
#	x-pack/plugins/monitoring/public/components/apm/instance/instance.js
#	x-pack/plugins/monitoring/public/components/beats/beat/beat.js
@chrisronline chrisronline added the backport This PR is a backport of another PR label Oct 1, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
monitoring 628 638 +10

async chunks size

id before after diff
monitoring 1.2MB 1.2MB +8.3KB

distributable file count

id before after diff
default 47593 47598 +5

page load bundle size

id before after diff
monitoring 183.5KB 222.8KB +39.4KB

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

@chrisronline chrisronline merged commit 45c215d into elastic:7.x Oct 1, 2020
@chrisronline chrisronline deleted the backport/7.x/pr-78208 branch October 1, 2020 18: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