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

[7.x] [Logs UI] Create screen to set up analysis ML jobs (#43413) #43782

Merged
merged 1 commit into from
Aug 23, 2019

Conversation

Zacqary
Copy link
Contributor

@Zacqary Zacqary commented Aug 22, 2019

Backports the following commits to 7.x:

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

* Add empty analysis tab

* Add ml capabilities check

* Add job status checking functionality

* Add a loading page for the job status check

* Change types / change method for deriving space ID / change setup requirement filtering check

* Use new structure

* Add module setup to log analysis jobs hook

* Add ID to path

* [Logs UI] Add analyis setup landing screen

* Add function to set up ML module on click

* Use partial type for start and end props

* Add start and end time selection

* Fix syntax

* Change seconds timestamp to ms

* Update wording

* Use FormControlLayout to clear datepickers

* Update wording about earlier start date

* Remove specific point in time wording

* Fix typechecking

* Reload analysis page on successful job creation

* Add error handling for setup failure

* Update description ton of feature to reflect 7.4 feature set

* Add toggleable default message

* Revert to EuiFormControlLayout until eui changes are pushed

* Remove sample data index if user has it set
@Kerry350 Kerry350 force-pushed the backport/7.x/pr-43413 branch from fd0821b to 6db2447 Compare August 23, 2019 17:53
@Kerry350
Copy link
Contributor

Kerry350 commented Aug 23, 2019

My bad, the force push it's mentioning from me is because I didn't notice the backport PR for this was open. I ran yarn backport --username zacqary for the backport, but it does a push before failing and telling you a PR already exists. I'll merge once it's done building again, then backport the results screen and finally the wrap up PR.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@Kerry350
Copy link
Contributor

jenkins retest this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@Zacqary Zacqary merged commit a8a0f35 into elastic:7.x Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants