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

[Logs UI] anomaly view cleanup #65046

Closed
5 tasks done
katrin-freihofner opened this issue May 4, 2020 · 6 comments
Closed
5 tasks done

[Logs UI] anomaly view cleanup #65046

katrin-freihofner opened this issue May 4, 2020 · 6 comments
Assignees
Labels
Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Milestone

Comments

@katrin-freihofner
Copy link
Contributor

katrin-freihofner commented May 4, 2020

This ticket summarizes a cleanup of the Anomalies/Log rate tab.

This is dependent on the new anomaly table and setup flyout

Goal

The outcome of this ticket should look similar to this

meeting preview

Todos

  • remove the "Analyzed .... log entries..." info
  • remove the white background behind the time picker so it sits directly on the gray background
  • remove the log rate chart and legend
  • the Analyze in ML button should be replaced to trigger the setup flyout ([Logs UI] Move ML job setup to flyout #64754)
  • remove all the stats next to the charts

Please reach out anytime if something is unclear.

@katrin-freihofner katrin-freihofner added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label May 4, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@weltenwort weltenwort added the Feature:Logs UI Logs UI feature label May 4, 2020
@weltenwort
Copy link
Member

looks nice and clean 🧹

the Analyze in ML button should be replaced to trigger the setup flyout

So we don't want to link to the ML UI anymore? Or did you mean to say that the "Reconfigure job" button should be replaced?

@katrin-freihofner
Copy link
Contributor Author

Very good question. I think it's probably more interesting to look at specific anomalies in ML. Besides that, I'm wondering if we can link somewhere meaningful - we are going to have categories and log rate anomalies there.

@weltenwort
Copy link
Member

True, there's no one link once we have multiple anomaly types in there. We already have per-row "Analyze in ML" links that depend on the specific anomaly type.

Is there a downside to keeping the link until we're at the point that we're showing the anomalies from multiple jobs?

@katrin-freihofner
Copy link
Contributor Author

I don't see one, I'm perfectly fine with keeping it for now.

@sgrodzicki
Copy link
Contributor

#64755

@zube zube bot added [zube]: Done and removed [zube]: Ready labels Jul 13, 2020
@zube zube bot removed the [zube]: Done label Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

No branches or pull requests

6 participants