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

TermsGroupSourceTests.testSupportsIncrementalBucketUpdate is flaky #60794

Closed
rjernst opened this issue Aug 5, 2020 · 1 comment · Fixed by #60807
Closed

TermsGroupSourceTests.testSupportsIncrementalBucketUpdate is flaky #60794

rjernst opened this issue Aug 5, 2020 · 1 comment · Fixed by #60807
Assignees
@hendrikmuhs
Labels
:ml/Transform Transform >test-failure Triaged test failures from CI

Comments

@rjernst
Copy link
Member

rjernst commented Aug 5, 2020 •
edited
Loading

Build scan:
https://gradle-enterprise.elastic.co/s/et6j32wppwsqk

Repro line:
./gradlew ':x-pack:plugin:core:test' --tests "org.elasticsearch.xpack.core.transform.transforms.pivot.TermsGroupSourceTests.testSupportsIncrementalBucketUpdate" -Dtests.seed=8ED60956387B79DC -Dtests.security.manager=true -Dtests.locale=ca-ES -Dtests.timezone=America/Monterrey -Druntime.java=11

Reproduces locally?: yes

Applicable branches: master, 7.x

Failure excerpt:

  2> java.lang.AssertionError: expected:<false> but was:<true>
        at __randomizedtesting.SeedInfo.seed([8ED60956387B79DC:EA838C26671EEB47]:0)
        at org.junit.Assert.fail(Assert.java:88)
        at org.junit.Assert.failNotEquals(Assert.java:834)
        at org.junit.Assert.assertEquals(Assert.java:118)
        at org.junit.Assert.assertEquals(Assert.java:144)
        at org.elasticsearch.xpack.core.transform.transforms.pivot.TermsGroupSourceTests.testSupportsIncrementalBucketUpdate(TermsGroupSourceTests.java:53)

I encountered this test failing in a PR, and found it reproduced on master, inconsistently. I ran with 20 iterations and got 8 failures.
@rjernst rjernst added the >test-failure Triaged test failures from CI label Aug 5, 2020
rjernst added a commit that referenced this issue Aug 5, 2020
@rjernst
Mute flaky transform pivot test
4483108 
see #60794
rjernst added a commit that referenced this issue Aug 5, 2020
@rjernst
Mute flaky transform pivot test
d88098c 
see #60794
@hendrikmuhs hendrikmuhs added the :ml/Transform Transform label Aug 6, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml/Transform)

@hendrikmuhs hendrikmuhs self-assigned this Aug 6, 2020
hendrikmuhs pushed a commit to hendrikmuhs/elasticsearch that referenced this issue Aug 6, 2020
remove test, scripts are excluded in the change collector, the test is a
697ccb2 
leftover from a previous solution of elastic#57332, which has been discarded

relates elastic#60724
fixes elastic#60794
@hendrikmuhs hendrikmuhs mentioned this issue Aug 6, 2020
Merged
hendrikmuhs pushed a commit that referenced this issue Aug 6, 2020
[Transform] remove wrong test (#60807)
596ea05 
remove test, scripts are excluded in the change collector, the test is a leftover from a previous
solution of #57332, which has been discarded

relates #60724
fixes #60794
hendrikmuhs pushed a commit that referenced this issue Aug 6, 2020
[Transform] remove wrong test (#60807)
b210aaf 
remove test, scripts are excluded in the change collector, the test is a leftover from a previous
solution of #57332, which has been discarded

relates #60724
fixes #60794
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hendrikmuhs hendrikmuhs

Labels
:ml/Transform Transform >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Transform] remove wrong test hendrikmuhs/elasticsearch
3 participants
@rjernst @hendrikmuhs @elasticmachine