-
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] SplitIndexIT testCreateSplitIndexToN failing #92183
Comments
Pinging @elastic/es-data-management (Team:Data Management) |
The test timed out, but it seems to me that these exceptions are relevant:
|
Looking at the failure history, I only see this happening on windows boxes. I'm not sure how this could be related to our test -- maybe something in the windows build environment? |
I'm not to the bottom of this yet, but thought I'd write up what I've learned so far: |
Here is the code that leads to file closure (several async calls):
And here is the code that leads to file deletes (also several async calls):
|
Since this appears to be internal to the splitting logic, I'm turning it over to the distributed team. Let me know if you disagree. |
Pinging @elastic/es-distributed (Team:Distributed) |
Failed again today: Test failed on Windows again |
Failed 5 times in the last 7 days, which seems quite a lot. This mornings failure: https://gradle-enterprise.elastic.co/s/wskoo2giniur2/tests/:server:internalClusterTest/org.elasticsearch.action.admin.indices.create.SplitIndexIT/testCreateSplitIndexToN |
I'll mute it. |
Thank @masseyke. This all seems to point to a long known issue with Windows:
@DaveCTurner seem to have spent some time getting to the bottom of this, among others. Actually the test under this one, i.e. |
Happened again on 8.6: https://gradle-enterprise.elastic.co/s/s5a3nqhkhauhg Probably missing backport of above fix on 8.6. |
The forgotten cherry-pick of elastic#93517. Closes elastic#92183
Build scan:
https://gradle-enterprise.elastic.co/s/bvb5puxxlbaou/tests/:server:internalClusterTest/org.elasticsearch.action.admin.indices.create.SplitIndexIT/testCreateSplitIndexToN
Reproduction line:
Applicable branches:
8.6
Reproduces locally?:
Didn't try
Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.action.admin.indices.create.SplitIndexIT&tests.test=testCreateSplitIndexToN
Failure excerpt:
The text was updated successfully, but these errors were encountered: