Skip to content
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

[APM] Anomaly detection: Display Anomaly detection option in the APM header #70440

Closed
formgeist opened this issue Jul 1, 2020 · 1 comment · Fixed by #71229
Closed

[APM] Anomaly detection: Display Anomaly detection option in the APM header #70440

formgeist opened this issue Jul 1, 2020 · 1 comment · Fixed by #71229
Assignees
Labels
apm:ml Integration between APM and ML Team:APM All issues that need APM UI Team support v7.9.0

Comments

@formgeist
Copy link
Contributor

formgeist commented Jul 1, 2020

Summary

Related to elastic/apm#280 and #69479

We want to display an option in our general APM header that will allow the user to go and setup Anomaly detection easily from anywhere in the UI. We will additionally increase visibility for users who have the available capabilities in their license to enable the integration immediately and get the value expected from it e.g. on the Service maps view.

Solution

Show an Anomaly detection option in the general APM header.

Anomaly detection has yet to be enabled

We will make the option especially visible when the Anomaly detection feature is yet to be enabled by two criteria;

  1. No anomaly detection jobs have been found for any environment in the data
  2. No job available for the selected environment e.g. user selected staging we'll display a highlighted icon next to the button to draw attention to it.

Slice 1

Additionally, we would like to show a tooltip on hover to explain the highlighted icon. The text will be dynamic to display either an all or specific environment message.

Screenshot 2020-07-01 at 13 50 40

Screenshot 2020-07-01 at 13 50 44

Anomaly detection is already enabled for environment(s) or license privileges are insufficient.

If there are jobs enabled for....

  • the currently selected environment
  • or any of the environments when All is selected

we will display the default button icon state.

Slice 2

The default state will also be shown when the license privileges do not allow for the integration to be enabled unless they start a trial. The start trial message will appear in the Settings view as described in [add a link to the issue – @formgeist ]

@formgeist formgeist added Team:APM All issues that need APM UI Team support v7.9.0 apm:ml Integration between APM and ML labels Jul 1, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

ogupte added a commit to ogupte/kibana that referenced this issue Jul 9, 2020
ogupte added a commit that referenced this issue Jul 13, 2020
…71229)

* Closes #70440 by adding a setup link to anomaly detection setting in the home header

* PR feedback and type error fix

* Code cleanup and PR feedback

* Modified getEnvironmentUiFilterES return type from `ESFilter | undefined` to `ESFilter[]` for ease of use.

Co-authored-by: Elastic Machine <[email protected]>
ogupte added a commit to ogupte/kibana that referenced this issue Jul 13, 2020
…lastic#71229)

* Closes elastic#70440 by adding a setup link to anomaly detection setting in the home header

* PR feedback and type error fix

* Code cleanup and PR feedback

* Modified getEnvironmentUiFilterES return type from `ESFilter | undefined` to `ESFilter[]` for ease of use.

Co-authored-by: Elastic Machine <[email protected]>
ogupte added a commit that referenced this issue Jul 13, 2020
…71229) (#71459)

* Closes #70440 by adding a setup link to anomaly detection setting in the home header

* PR feedback and type error fix

* Code cleanup and PR feedback

* Modified getEnvironmentUiFilterES return type from `ESFilter | undefined` to `ESFilter[]` for ease of use.

Co-authored-by: Elastic Machine <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:ml Integration between APM and ML Team:APM All issues that need APM UI Team support v7.9.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants