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] Move ML job setup to flyout #64754

Closed
katrin-freihofner opened this issue Apr 29, 2020 · 6 comments · Fixed by #68366
Closed

[Logs UI] Move ML job setup to flyout #64754

katrin-freihofner opened this issue Apr 29, 2020 · 6 comments · Fixed by #68366
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

We want to move the existing ML job setups to a flyout. This will prepare us for the upcoming merge of (log rate and category) anomalies to one overview and is also more consistent compared to our current solution.

The new setup should work like this:

ml-log-rate-setup

This means the following changes:

  • if ML is not yet set up, both tabs (log rate and categories) should show an empty state and open the flyout with the setup screen

  • all warning/error/success states can stay in the setup as we have them right now

  • Please ignore the button leading back from the setup to the job list for now - we will need this once we can provide the ML job list

Design issue
Figma prototype
Figma design file

@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 Apr 29, 2020
@elasticmachine
Copy link
Contributor

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

@jasonrhodes
Copy link
Member

@afgomez can you explain why this is currently blocked here in the comments? Thanks!

@afgomez
Copy link
Contributor

afgomez commented Jun 15, 2020

@jasonrhodes sure.

The content for the initial setup screen is yet to be defined. Without content the flyout in the initial setup will fly over an empty screen.

@katrin-freihofner
Copy link
Contributor Author

What is missing is the case where no ML jobs are set up but the user has a premium license (or free trial). For the other cases, we have a solution #64921

@mukeshelastic
Copy link

mukeshelastic commented Jun 18, 2020

Just to make sure I understood this correctly, the content we are looking for here is the one that is displayed on the main screen when the user clicks on anomalies and categories when no ML jobs have been set up for them and the user has the correct license ( or free valid ongoing trial).

If that's what we are looking for then.. here is a possible content.. We can run this by docs folks for grammar later..

" To show log <anomalies/categories>, machine learning jobs need to be set up." Button with a link to flyout is shown and the button text says: "ML setup"

@afgomez
Copy link
Contributor

afgomez commented Jun 24, 2020

Just to make sure I understood this correctly, the content we are looking for here is the one that is displayed on the main screen when the user clicks on anomalies and categories when no ML jobs have been set up for them and the user has the correct license

That is correct :)

I'll go ahead and write the content

@zube zube bot added [zube]: Done and removed [zube]: Ready labels Jul 6, 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

Successfully merging a pull request may close this issue.

6 participants