-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[BUG] :test:logger-usage:test failure flakey test #2893
Comments
@davidlago Could you look into this? |
Stack trace related to JDK Bug 8254876, fix in JDK16 just throws the NPE earlier. Cause is passing a null as OpenSearch/libs/core/src/main/java/org/opensearch/common/io/PathUtils.java Lines 62 to 74 in ea1cc9d
This argument is OpenSearch/test/framework/src/main/java/org/opensearch/bootstrap/BootstrapForTesting.java Lines 133 to 135 in ea1cc9d
But this should return OpenSearch/server/src/main/java/org/opensearch/common/Strings.java Lines 125 to 127 in ea1cc9d
OpenSearch/server/src/main/java/org/opensearch/common/Strings.java Lines 101 to 103 in ea1cc9d
|
Failed to reproduce in 1000 test runs. The only possible source of failure is a change in the system property between the null check (line 133) and retrieving the property again (line 134). Submitted #5645 to save the initial property value to ensure the null check is of the same String being passed later. |
Describe the bug
:test:logger-usage:test failure
Appeared in
#2726 (comment)
#1096 (comment)
The text was updated successfully, but these errors were encountered: