-
Notifications
You must be signed in to change notification settings - Fork 25.2k
[CI] BWC issue unknown setting [index.time_series.start_time] #89171
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
Comments
I have fixed it in #89236 |
@salvatore-campagna can you help to merge the PR #89236, to fix the failed test |
Pinging @elastic/es-analytics-geo (Team:Analytics) |
I merged #89236 to mute the repeated test failures. That might be the long-term fix too, but maybe not. The failures seem to be caused by the changes of #88695, in particular the addition of I will leave it up to @elastic/es-analytics-geo to decide if that's acceptable. If it is then this issue can be closed, otherwise an extra test that works against 8.0 might be needed. |
@droberts195, your root cause analysis is spot on. The fix should close this issue. Thanks again for taking care of this |
Stops the repeated test failures due to elastic#89171
CI Link
https://gradle-enterprise.elastic.co/s/uapcsmavbl7fg
Repro line
./gradlew ':qa:mixed-cluster:v8.0.1#mixedClusterTest' -Dtests.class="org.elasticsearch.backwards.MixedClusterClientYamlTestSuiteIT" -Dtests.method="test {p0=field_caps/40_time_series/Get time series field caps with conflicts}" -Dtests.seed=124BAB58491BC3E1 -Dtests.bwc=true -Dtests.locale=zh-Hant-TW -Dtests.timezone=Pacific/Niue -Druntime.java=17
Does it reproduce?
Yes
Applicable branches
main
Failure history
No response
Failure excerpt
The text was updated successfully, but these errors were encountered: