-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Stack Monitoring] add missing mb field #112251
[Stack Monitoring] add missing mb field #112251
Conversation
Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui) |
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those +1,178,984 −240,770 diffs. They'll get you every time :)
Let me know if there are any testing steps you'd recommend, otherwise LGTM.
* add missing mb field * fix tests
* add missing mb field * fix tests
Summary
Fixes #110719
The bug was introduced in #96205 for 7.14, causing the ES query to be incorrect as we were missing the field in one of the aggregations:
Reviewer notes
Checklist
Delete any items that are not applicable to this PR.