-
Notifications
You must be signed in to change notification settings - Fork 25k
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 #95603
Comments
Pinging @elastic/es-core-infra (Team:Core/Infra) |
I'm thinking this is enterprise search becoming available (congrats!) due to this line:
Can y'all take a look? Checkout out rest-api/usage.asciidoc for more info. |
Another one today: https://gradle-enterprise.elastic.co/s/jntfw5cm3pgq2 I tried reproducing this and was a bit surprised I couldn't. I'd be interested if this works for anybody else looking at this and maybe tell me what I did wrong or if this really doesn't cleanly reproduce. I used:
On the current 8.8 branch and that passed locally. |
It must be the |
This specific test doesn't run on snapshot build so it's expected: Although it was supposed to be temporary so I wonder if we just forgot to re-activate the test for all builds @ywangd ? |
Pinging @elastic/ent-search-eng (Team:Enterprise Search) |
The previous fix (elastic#95565) didn't work since the section was misplaced. Note that this test runs only on snapshot build so I tested manually and the failure is now related to remote_clusters section missing. Closes elastic#95603
The doc test was disabled because of the remote cluster security project that is being developed behind a feature flag. We were initially expecting to remove the feature flag in 8.8. But that slipped and we are now targeting 8.9. Sorry the inconvenience. Other than disabling the doc test for snapshot build, I don't see other viable ways to make it work without showing unreleased content in the Doc. Please let me know if you have other suggestions. Again, sorry this is causing troubles. |
The previous fix (elastic#95565) didn't work since the section was misplaced. Note that this test runs only on snapshot build so I tested manually and the failure is now related to remote_clusters section missing. Closes elastic#95603
Thanks for explaining @ywangd , the fix is merged now so it should continue to work when the feature flag is removed. |
Failed 7 times in the past 3 days:
https://gradle-enterprise.elastic.co/scans/tests?search.timeZoneId=Europe/Berlin&tests.container=org.elasticsearch.smoketest.DocsClientYamlTestSuiteIT&tests.test=test%20%7Byaml%3Dreference/rest-api/usage/line_38%7D
Build scan:
https://gradle-enterprise.elastic.co/s/bpdzoghroz5vq/tests/:docs:yamlRestTest/org.elasticsearch.smoketest.DocsClientYamlTestSuiteIT/test%20%7Byaml=reference%2Frest-api%2Fusage%2Fline_38%7D
Reproduction line:
Applicable branches:
main, 8.8
Reproduces locally?:
Didn't try
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: