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

[DOCS] Skips testing for forecast API #31439

Closed
wants to merge 2 commits into from

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Jun 19, 2018

This PR skips code snippet testing for the machine learning forecast API, since (even though it works from the console with basic farequote data), the gradle checks fail as follows:

1> [2018-06-18T20:45:24,050][INFO ][o.e.s.XDocsClientYamlTestSuiteIT] Stash dump on test failure [{
1> "stash" : {
1> "body" : {
1> "error" : {
1> "root_cause" : [
1> {
1> "type" : "status_exception",
1> "reason" : "Cannot run forecast: Forecast cannot be executed as job requires data to have been processed and modeled",
1> "stack_trace" : "ElasticsearchStatusException[Cannot run forecast: Forecast cannot be executed as job requires data to have been processed and modeled]

I suspect the CI is too quick and the setup data has not been modelled yet.

@lcawl lcawl added >docs General docs changes v7.0.0 :ml Machine learning v6.4.0 v6.3.1 labels Jun 19, 2018
@lcawl lcawl requested a review from hendrikmuhs June 19, 2018 17:15
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@hendrikmuhs
Copy link
Contributor

I run into a similar issue recently, in automation things are simply to quick. I think if you flush the job prior forecasting it should start working, something like:

  - do:
      xpack.ml.flush_job:
        job_id: "total-requests"

@lcawl
Copy link
Contributor Author

lcawl commented Jun 19, 2018

@hendrikmuhs Unfortunately that doesn't seem to do the trick. If you don't have any other suggestions, I'll just disable these tests.

@hendrikmuhs
Copy link
Contributor

@lcawl

I checked again: The problem is the datafeed. Unfortunately I could not find a way to wait for the completion of the datafeed. I missed that flush is not working for this case, sorry, my mistake.

There is 1 more possibility, if you use xpack.ml.post_data instead of a data feed, which means directly uploading data into a job by-passing indexing and data-feeding and again wait for the flush.

But: I guess this would be a big change, likely affecting other parts or you need a separate jobs to test forecasting.

I do not know if it's worth the effort, please judge yourself, I am fine with disabling the test if you prefer.

@lcawl
Copy link
Contributor Author

lcawl commented Jun 22, 2018

@hendrikmuhs I unfortunately encountered the same problem with posting data, so I will leave this test disabled. Thanks for the suggestions!

@lcawl lcawl closed this Jun 22, 2018
@lcawl lcawl deleted the lcawley-forecast-API branch June 22, 2018 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes :ml Machine learning v6.3.1 v6.4.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants