Closed romseygeek closed 6 months ago
Pinging @elastic/ml-core (Team:ML)
Pinging @elastic/es-core-features (Team:Core/Features)
I think #73734 would fix this. As an interim fix we should probably just ignore this warning.
A related failure in docs yaml tests: https://gradle-enterprise.elastic.co/s/g2vrkxovl2ws2
[2564-07-08T15:10:11,580][INFO ][o.e.s.DocsClientYamlTestSuiteIT] [test] [yaml=reference/query-dsl/exists-query/line_56] before test |
-- | --
| ก.ค. ๐๘, ๒๕๖๔ ๓:๑๐:๑๑ หลังเที่ยง org.elasticsearch.client.RestClient logResponse |
| WARNING: request [GET http://[::1]:43489/_search?error_trace=true] returned 1 warnings: [299 Elasticsearch-7.15.0-SNAPSHOT-a049585f298d10ed58c64ecac6902df5c4b86280 "this request accesses system indices: [.async-search], but in a future major version, direct access to system indices will be prevented by default"] |
| [
I think this is an evolution of #74141. After fixing some of the race conditions between ML job open, close and kill the races move on to code that runs later.
In the failure from this issue's description the relevant bit of the server side log is this:
[2021-07-07T14:19:35,848][INFO ][o.e.x.m.MachineLearning ] [yamlRestTest-0] Starting machine learning feature reset
[2021-07-07T14:19:35,850][INFO ][o.e.x.m.j.p.a.AutodetectProcessManager] [yamlRestTest-0] [set-upgrade-mode-job] Loading model snapshot [N/A], job latest_record_timestamp [N/A]
[2021-07-07T14:19:35,919][INFO ][o.e.x.m.a.TransportKillProcessAction] [yamlRestTest-0] [set-upgrade-mode-job] Killing job
[2021-07-07T14:19:35,919][TRACE][o.e.x.m.j.p.a.AutodetectProcessManager] [yamlRestTest-0] [set-upgrade-mode-job] Killing process: awaitCompletion = [true]; reason = [kill process (api)]
[2021-07-07T14:19:35,975][INFO ][o.e.c.m.MetadataDeleteIndexService] [yamlRestTest-0] [.ml-state-000001/wlkDnNKlQrqyEX_SoHKDcg] deleting index
[2021-07-07T14:19:35,975][INFO ][o.e.c.m.MetadataDeleteIndexService] [yamlRestTest-0] [.ml-notifications-000002/UdDUIn_vSGW4XpL0OXCb-w] deleting index
[2021-07-07T14:19:35,975][INFO ][o.e.c.m.MetadataDeleteIndexService] [yamlRestTest-0] [.ml-config/-HtKKmzATrqhExujwSN9BQ] deleting index
[2021-07-07T14:19:35,975][INFO ][o.e.c.m.MetadataDeleteIndexService] [yamlRestTest-0] [.ml-anomalies-shared/ZP7tVtBfQCeUXamDP4Tj7w] deleting index
[2021-07-07T14:19:35,976][INFO ][o.e.c.m.MetadataDeleteIndexService] [yamlRestTest-0] [.ml-annotations-6/lpA4DM6aRauiVpREJX9vhw] deleting index
[2021-07-07T14:19:36,130][INFO ][o.e.c.m.MetadataDeleteIndexService] [yamlRestTest-0] [airline-data/Zqw03jwRQFmmK-DuatKkGg] deleting index
[2021-07-07T14:19:36,266][INFO ][o.e.x.i.a.TransportPutLifecycleAction] [yamlRestTest-0] adding index lifecycle policy [.fleet-actions-results-ilm-policy]
[2021-07-07T14:19:36,318][INFO ][o.e.x.i.a.TransportPutLifecycleAction] [yamlRestTest-0] adding index lifecycle policy [synthetics]
[2021-07-07T14:19:36,368][INFO ][o.e.x.i.a.TransportPutLifecycleAction] [yamlRestTest-0] adding index lifecycle policy [.deprecation-indexing-ilm-policy]
[2021-07-07T14:19:36,392][INFO ][o.e.x.m.j.p.a.AutodetectProcessManager] [yamlRestTest-0] Closing job [set-upgrade-mode-job], because [job is already closing]
[2021-07-07T14:19:36,396][WARN ][o.e.x.m.j.p.a.o.AutodetectResultProcessor] [yamlRestTest-0] [set-upgrade-mode-job] some results not processed due to the process being killed
[2021-07-07T14:19:36,397][INFO ][o.e.x.m.j.p.a.AutodetectCommunicator] [yamlRestTest-0] [set-upgrade-mode-job] job closed
[2021-07-07T14:19:36,407][INFO ][o.e.c.m.MetadataCreateIndexService] [yamlRestTest-0] [.ml-config] creating index, cause [auto(update api)], templates [], shards [1]/[1]
[2021-07-07T14:19:36,408][INFO ][o.e.c.r.a.AllocationService] [yamlRestTest-0] updating number_of_replicas to [0] for indices [.ml-config]
[2021-07-07T14:19:36,472][INFO ][o.e.c.m.MetadataCreateIndexService] [yamlRestTest-0] [.ml-annotations-6] creating index, cause [api], templates [], shards [1]/[1]
[2021-07-07T14:19:36,473][INFO ][o.e.c.r.a.AllocationService] [yamlRestTest-0] updating number_of_replicas to [0] for indices [.ml-annotations-6]
Or, in plain English:
Another one just happened in a docs test: https://gradle-enterprise.elastic.co/s/q6lylbs4wg3we
However, this one is for the .async-search
index, not .ml-config
, and I don't think ML is responsible for this one.
The relevant bit of the server-side log is:
[2021-07-08T13:14:21,548][INFO ][o.e.c.m.MetadataCreateIndexService] [node-0] [.async-search] creating index, cause [api], templates [], shards [1]/[0]
[2021-07-08T13:14:21,636][INFO ][o.e.c.m.MetadataDeleteIndexService] [node-0] [library/wnvB3FnqThqM1VDYR62u4A] deleting index
[2021-07-08T13:14:21,637][INFO ][o.e.c.m.MetadataDeleteIndexService] [node-0] [.async-search/-C8hXOI_QOuU488GHnhuKQ] deleting index
[2021-07-08T13:14:21,704][INFO ][o.e.c.m.MetadataCreateIndexService] [node-0] [.async-search] creating index, cause [auto(bulk api)], templates [], shards [1]/[1]
[2021-07-08T13:14:21,819][INFO ][o.e.c.m.MetadataMappingService] [node-0] [.async-search/mkUDT9-hRDqacmjcBBo8aQ] create_mapping [_doc]
[2021-07-08T13:14:22,008][INFO ][o.e.c.m.MetadataCreateIndexService] [node-0] [sales] creating index, cause [api], templates [], shards [1]/[1]
[2021-07-08T13:14:22,180][INFO ][o.e.c.m.MetadataDeleteIndexService] [node-0] [.async-search/mkUDT9-hRDqacmjcBBo8aQ] deleting index
So it looks like the .async-search
index is being auto-created after it's been deleted after the previous test, then gets deleted again by the "delete all indices" cleanup code, yet this triggers a deprecation warning that fails the test.
This is another one that failed due to .async-search
being a system index: https://gradle-enterprise.elastic.co/s/udfnzxfgyd37k
In this case the test that failed was reference/query-dsl/match-query/line_219
. However, if you look in the full console log you can see that the test that ran immediately before was reference/sql/endpoints/rest/line_576
. That was added recently in #74975 and uses async search, so that is what caused the .async-search
index to be created at an unexpected moment. So this explains why these have started failing recently.
@williamrandolph do you think you would be able to merge #73734 soon with all the nice-to-haves deferred to a followup? I think that would be good enough for the docs tests. The only other solution would be to disable snippet testing for all the snippets added in #74975 that use async search.
/cc @jrodewig
I see a lot of failures in yaml tests this this "unexpected warning header" today, its one of the biggest sources of failures at the moment (although the overall frequency is bareable), so I'm +1 on gettin #73734 in as soon as @williamrandolph thinks its possible to get the noise level down.
Thanks for the ping here. #73734 is a better long-term fix, but I've merged #75237 to mute snippet tests and reduce noise in the meantime.
@droberts195 Thanks for the ping. I will bring PR #73734 up to date and merge it as soon as I can.
@williamrandolph are you still planning to merge #73734 to address this?
Yes, now that we've finished our features for 8.0, I need to take another pass at that PR. Thanks for the reminder.
Pinging @elastic/es-data-management (Team:Data Management)
I talked to @williamrandolph and between https://github.com/elastic/elasticsearch/pull/94130 and https://github.com/elastic/elasticsearch/pull/94191 we think this one's probably closable. I put up #106140 to revert https://github.com/elastic/elasticsearch/pull/75237, and once that's merged I'm going to close this ticket.
Build scan: https://gradle-enterprise.elastic.co/s/svd6gy2xzpfn4/tests/:x-pack:plugin:yamlRestTest/org.elasticsearch.xpack.test.rest.XPackRestIT/test%20%7Bp0=data_stream%2F140_data_stream_aliases%2FCreate%20data%20stream%20alias%7D
Reproduction line:
./gradlew ':x-pack:plugin:yamlRestTest' --tests "org.elasticsearch.xpack.test.rest.XPackRestIT.test {p0=data_stream/140_data_stream_aliases/Create data stream alias}" -Dtests.seed=9F9D630801B00349 -Dtests.locale=it-IT -Dtests.timezone=Asia/Ashkhabad -Druntime.java=8
Applicable branches: 7.x, master
Reproduces locally?: No
Failure history: https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.xpack.test.rest.XPackRestIT&tests.test=test%20%7Bp0%3Ddata_stream/140_data_stream_aliases/Create%20data%20stream%20alias%7D
Failure excerpt: