Skip to content

[7.8] [APM] Service maps - Fix missing ML status for services with jobs but no anomalies (#68486)#68735

Merged
ogupte merged 3 commits intoelastic:7.8from
ogupte:backport/7.8/pr-68486
Jun 10, 2020
Merged

[7.8] [APM] Service maps - Fix missing ML status for services with jobs but no anomalies (#68486)#68735
ogupte merged 3 commits intoelastic:7.8from
ogupte:backport/7.8/pr-68486

Conversation

@ogupte
Copy link
Copy Markdown
Contributor

@ogupte ogupte commented Jun 10, 2020

Backports the following commits to 7.8:

… no anomalies (elastic#68486)

* Closes elastic#68485 by:
  - use the ML plugin to query for all APM jobs
  - inspect the ml job groups to find all jobs related to a particular service
  - use the mlAnomalySearch client to get ml job buckets with the max anomaly score
  - query for the model_plot buckets to obtain actual/median values for the ML description
  - return the relevant ML job with the max anomaly score for a service
  - indicate to the user that no anomalies were found for a service with an ml job

* - Use the anomalyDetectorsProvider jobs API rather than the search endpoint directly
- Defines a specific return type for the ml jobs api
- Update the empty anomaly data message

* Code and types cleanup

* Return to using record result type on anomaly queries. These are the
same values used in the anomaly explorer and it includes actual &
typical values which greatly improve performance of the previous query.

* - If anomaly data is missing show a gray border around node

* - moved AnomalyDetection out of service map Contents into own component

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	x-pack/plugins/ml/server/shared.ts
@ogupte ogupte added the backport This PR is a backport of another PR label Jun 10, 2020
- uses existing mlSearch API rather than the mlAnomalySearch API
@ogupte
Copy link
Copy Markdown
Contributor Author

ogupte commented Jun 10, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

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

@ogupte ogupte merged commit 2dc4c8b into elastic:7.8 Jun 10, 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.

3 participants