Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[CI] PreallocateTests testPreallocate failing #110948

Closed
elasticsearchmachine opened this issue Jul 16, 2024 · 3 comments · Fixed by #110998
Closed

[CI] PreallocateTests testPreallocate failing #110948

elasticsearchmachine opened this issue Jul 16, 2024 · 3 comments · Fixed by #110998
Assignees
@rjernst
Labels
:Core/Infra/Core Core issues without another label low-risk An open issue or test failure that is a low risk to future releases Team:Core/Infra Meta label for core/infra team >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

Build Scans:

Reproduction Line:

gradlew ":libs:elasticsearch-native:test" --tests "org.elasticsearch.nativeaccess.PreallocateTests.testPreallocate" -Dtests.seed=2CBD5DEA907DA216 -Dtests.locale=sr-BA -Dtests.timezone=America/Vancouver -Druntime.java=22

Applicable branches:
main

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: 
Expected: <1048576L>
     but: was <0L>

Issue Reasons:

  • [main] 5 failures in test testPreallocate (17.9% fail rate in 28 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Core/Infra/Core Core issues without another label >test-failure Triaged test failures from CI labels Jul 16, 2024
elasticsearchmachine added a commit that referenced this issue Jul 16, 2024
@elasticsearchmachine
Mute org.elasticsearch.nativeaccess.PreallocateTests testPreallocate #…
774f1a4 
…110948
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 5 failures in test testPreallocate (17.9% fail rate in 28 executions)

Build Scans:

@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team needs:risk Requires assignment of a risk label (low, medium, blocker) labels Jul 16, 2024
@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch 8.15

Mute Reasons:

  • [8.15] 3 consecutive failures in step encryption-at-rest
  • [8.15] 5 failures in test testPreallocate (5.9% fail rate in 85 executions)
  • [8.15] 3 failures in step encryption-at-rest (100.0% fail rate in 3 executions)
  • [8.15] 3 failures in pipeline elasticsearch-periodic (100.0% fail rate in 3 executions)
  • [8.15] 2 failures in pipeline elasticsearch-pull-request (28.6% fail rate in 7 executions)

Build Scans:

elasticsearchmachine added a commit that referenced this issue Jul 16, 2024
@elasticsearchmachine
Mute org.elasticsearch.preallocate.PreallocateTests testPreallocate #…
107eaa7 
…110948
salvatore-campagna pushed a commit to salvatore-campagna/elasticsearch that referenced this issue Jul 17, 2024
@elasticsearchmachine @salvatore-campagna
Mute org.elasticsearch.nativeaccess.PreallocateTests testPreallocate e…
407a733 
…lastic#110948
rjernst added a commit to rjernst/elasticsearch that referenced this issue Jul 17, 2024
@rjernst
Skip preallocate tests on windows
6d15eaf 
The preallocate tests assumed that preallocation was using the fallback
implementation which calls setLength on Windows. However, that fallback
only happens inside the SharedBytes class, so windows doesn't actually
do anything when tryPreallocate is called. This commit skips the test on
windows.

closes elastic#110948
@rjernst rjernst mentioned this issue Jul 17, 2024
Merged
@rjernst rjernst self-assigned this Jul 18, 2024
@rjernst rjernst added low-risk An open issue or test failure that is a low risk to future releases and removed needs:risk Requires assignment of a risk label (low, medium, blocker) labels Jul 18, 2024
rjernst added a commit that referenced this issue Jul 19, 2024
@rjernst
Skip preallocate tests on windows (#110998)
d21d424 
The preallocate tests assumed that preallocation was using the fallback
implementation which calls setLength on Windows. However, that fallback
only happens inside the SharedBytes class, so windows doesn't actually
do anything when tryPreallocate is called. This commit skips the test on
windows.

closes #110948
rjernst added a commit to rjernst/elasticsearch that referenced this issue Jul 19, 2024
@rjernst
Skip preallocate tests on windows (elastic#110998)
ced19c4 
The preallocate tests assumed that preallocation was using the fallback implementation which calls setLength on Windows. However, that fallback only happens inside the SharedBytes class, so windows doesn't actually do anything when tryPreallocate is called. This commit skips the test on windows.

closes elastic#110948
@rjernst rjernst mentioned this issue Jul 19, 2024
Merged
elasticsearchmachine pushed a commit that referenced this issue Jul 19, 2024
@rjernst
Skip preallocate tests on windows (#110998) (#111100)
bdb1028 
The preallocate tests assumed that preallocation was using the fallback implementation which calls setLength on Windows. However, that fallback only happens inside the SharedBytes class, so windows doesn't actually do anything when tryPreallocate is called. This commit skips the test on windows.

closes #110948
ioanatia pushed a commit to ioanatia/elasticsearch that referenced this issue Jul 22, 2024
@rjernst @ioanatia
Skip preallocate tests on windows (elastic#110998)
9b23451 
The preallocate tests assumed that preallocation was using the fallback
implementation which calls setLength on Windows. However, that fallback
only happens inside the SharedBytes class, so windows doesn't actually
do anything when tryPreallocate is called. This commit skips the test on
windows.

closes elastic#110948
salvatore-campagna pushed a commit to salvatore-campagna/elasticsearch that referenced this issue Jul 23, 2024
@rjernst @salvatore-campagna
Skip preallocate tests on windows (elastic#110998)
766102e 
The preallocate tests assumed that preallocation was using the fallback
implementation which calls setLength on Windows. However, that fallback
only happens inside the SharedBytes class, so windows doesn't actually
do anything when tryPreallocate is called. This commit skips the test on
windows.

closes elastic#110948
@lkts lkts mentioned this issue Aug 13, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rjernst rjernst

Labels
:Core/Infra/Core Core issues without another label low-risk An open issue or test failure that is a low risk to future releases Team:Core/Infra Meta label for core/infra team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Skip preallocate tests on windows rjernst/elasticsearch
2 participants
@rjernst @elasticsearchmachine