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] Test Failure in org.elasticsearch.script.ScriptServiceTests.testContextCacheStats #54324

Closed
original-brownbear opened this issue Mar 27, 2020 · 4 comments · Fixed by #54356
Closed

[CI] Test Failure in org.elasticsearch.script.ScriptServiceTests.testContextCacheStats #54324

original-brownbear opened this issue Mar 27, 2020 · 4 comments · Fixed by #54356
Assignees
@stu-elastic
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test-failure Triaged test failures from CI

Comments

@original-brownbear
Copy link
Member

This just failed on CI, can't reproduce this locally though and haven't seen it before so not muting yet.

org.elasticsearch.script.ScriptServiceTests > testContextCacheStats FAILED
    junit.framework.AssertionFailedError: Expected exception GeneralScriptException but no exception was thrown
        at __randomizedtesting.SeedInfo.seed([4CB9D6CB3156F6BC:F21F3A8B0A50B9A]:0)
        at org.apache.lucene.util.LuceneTestCase.expectThrows(LuceneTestCase.java:2743)
        at org.apache.lucene.util.LuceneTestCase.expectThrows(LuceneTestCase.java:2733)
        at org.elasticsearch.script.ScriptServiceTests.testContextCacheStats(ScriptServiceTests.java:287)
REPRODUCE WITH: ./gradlew ':server:test' --tests "org.elasticsearch.script.ScriptServiceTests.testContextCacheStats" -Dtests.seed=4CB9D6CB3156F6BC -Dtests.security.manager=true -Dtests.locale=sr-Latn -Dtests.timezone=Africa/Windhoek -Dcompiler.java=13
@original-brownbear original-brownbear added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test-failure Triaged test failures from CI labels Mar 27, 2020
@elasticmachine
Copy link
Collaborator

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

@stu-elastic stu-elastic self-assigned this Mar 27, 2020
@stu-elastic
Copy link
Contributor

Taking a look, I just added this yesterday.

@stu-elastic
Copy link
Contributor

Muted for now.

@stu-elastic
Copy link
Contributor

Issue was randomly selecting two contexts that were the same, so rate limit was being overridden.

stu-elastic added a commit to stu-elastic/elasticsearch that referenced this issue Mar 27, 2020
@stu-elastic
Scripting: Use randomValueOtherThan in stats test
8e9cc3f 
Fixes: elastic#54324
@stu-elastic stu-elastic mentioned this issue Mar 27, 2020
Merged
stu-elastic added a commit that referenced this issue Mar 27, 2020
@stu-elastic
Scripting: Use randomValueOtherThan in stats test (#54356)
26cd243 
Fixes: #54324
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stu-elastic stu-elastic

Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Scripting: Use randomValueOtherThan in stats test stu-elastic/elasticsearch
3 participants
@original-brownbear @elasticmachine @stu-elastic