-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
[CI] DocsClientYamlTestSuiteIT test {yaml=reference/rest-api/usage/line_38} failing due to invalid script in pipeline #83038
Comments
Pinging @elastic/ml-core (Team:ML) |
The automatic Failure excerpt is not very readable here, this is the stack with the failing script:
|
This is not ML related. Something created a pipeline with a ML only shows it because we build the pipelines that are stored to extract specific information around ML related processors. We wrap the pipeline building in I searched the code base and found:
So, there is a scripting doc that references this invalid script it seems. But, I am not sure who is creating that |
The elasticsearch/docs/reference/ingest.asciidoc Lines 809 to 837 in 3d1cb49
That also defines elasticsearch/docs/reference/scripting/apis/create-stored-script-api.asciidoc Lines 12 to 21 in 3d1cb49
So I think there are two things here:
@jrodewig it looks like you refactored these docs most recently, albeit 9 months ago. I'm not sure why this problem didn't come up during that 9 months but please could you have a look. |
Pinging @elastic/es-docs (Team:Docs) |
Going back a bit in the test failure history, I see the compile failure already a few times in the last month |
Thanks for the ping and apologies for the noise. I'll fix up the invalid script and rename the ingest script to avoid naming collisions. |
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971)
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971)
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971)
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971)
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971)
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971)
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971)
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to #83038 (cherry picked from commit b413971) # Conflicts: # docs/reference/ingest.asciidoc
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038 (cherry picked from commit 3845a41)
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038 (cherry picked from commit 3845a41)
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038 (cherry picked from commit 3845a41)
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038 (cherry picked from commit 3845a41)
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038 (cherry picked from commit 3845a41)
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038 (cherry picked from commit 3845a41)
Changes: * Updates the example Painless script to be valid and aligns it with the example in [How to write a script](https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-scripting-using.html#script-stored-scripts). * Adds a hidden snippets to delete the script for cleanup. Relates to #83038 (cherry picked from commit 3845a41)
Changes: * Renames the example stored script to avoid naming collisions with the [stored script API docs](https://www.elastic.co/guide/en/elasticsearch/reference/master/create-stored-script-api.html). * Adds a hidden snippet to delete the script for cleanup. Relates to elastic/elasticsearch#83038 (cherry picked from commit b413971)
I cannot reproduce this on the commit on master locally, maybe a one-of error?
This comes from docs calling "xpack/_usage" but from the stack trace its the TransportGetTrainedModelsStatsAction that is calling using the failing script, so tagging ML for inspection
Build scan:
https://gradle-enterprise.elastic.co/s/q5ypvd5xlnuey/tests/:docs:integTest/org.elasticsearch.smoketest.DocsClientYamlTestSuiteIT/test%20%7Byaml=reference%2Frest-api%2Fusage%2Fline_38%7D
Reproduction line:
./gradlew ':docs:integTest' --tests "org.elasticsearch.smoketest.DocsClientYamlTestSuiteIT.test {yaml=reference/rest-api/usage/line_38}" -Dtests.seed=6A098B0603E56D9F -Dtests.locale=ga-IE -Dtests.timezone=America/Kralendijk -Druntime.java=17
Applicable branches:
master
Reproduces locally?:
No
Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.smoketest.DocsClientYamlTestSuiteIT&tests.test=test%20%7Byaml%3Dreference/rest-api/usage/line_38%7D
Failure excerpt:
The text was updated successfully, but these errors were encountered: