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] testThousandsAndPattern test fails on 7.x #72883

Closed
przemekwitek opened this issue May 10, 2021 · 3 comments
Closed

[CI] testThousandsAndPattern test fails on 7.x #72883

przemekwitek opened this issue May 10, 2021 · 3 comments
Assignees
@nik9000
Labels
Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI

Comments

@przemekwitek
Copy link
Contributor

Build scan:

https://gradle-enterprise.elastic.co/s/tlqehodl7gpgk

Repro line:

REPRODUCE WITH: ./gradlew ':server:test' --tests "org.elasticsearch.common.regex.RegexTests.testThousandsAndPattern" \
  -Dtests.seed=704ED4D78C990DFA \
  -Dtests.locale=ar-YE \
  -Dtests.timezone=Europe/Vienna \
  -Druntime.java=11

REPRODUCE WITH: ./gradlew ':server:test' --tests "org.elasticsearch.common.regex.RegexTests.testThousandsAndPattern" \
  -Dtests.seed=704ED4D78C990DFA \
  -Dtests.locale=ar-YE \
  -Dtests.timezone=Europe/Vienna \
  -Druntime.java=11

Reproduces locally?:

Yes:

org.elasticsearch.common.regex.RegexTests > testThousandsAndPattern FAILED
    java.lang.AssertionError
        at __randomizedtesting.SeedInfo.seed([704ED4D78C990DFA:1C9B8B5E2D21CA03]:0)
        at org.junit.Assert.fail(Assert.java:86)
        at org.junit.Assert.fail(Assert.java:95)
        at org.elasticsearch.common.regex.RegexTests.testThousandsAndPattern(RegexTests.java:148)

Applicable branches:

7.x

Failure history:

This is the first failure I could find.

Failure excerpt:

14:12:27 org.elasticsearch.common.regex.RegexTests > testThousandsAndPattern FAILED
14:12:27     java.lang.AssertionError
14:12:27         at __randomizedtesting.SeedInfo.seed([704ED4D78C990DFA:1C9B8B5E2D21CA03]:0)
14:12:27         at org.junit.Assert.fail(Assert.java:86)
14:12:27         at org.junit.Assert.fail(Assert.java:95)
14:12:27         at org.elasticsearch.common.regex.RegexTests.testThousandsAndPattern(RegexTests.java:148)
@przemekwitek przemekwitek added >test-failure Triaged test failures from CI Team:Data Management Meta label for data/management team labels May 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@danhermann danhermann added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) and removed Team:Data Management Meta label for data/management team labels May 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@nik9000 nik9000 self-assigned this May 10, 2021
@nik9000
Copy link
Member

nik9000 commented May 10, 2021

Looks like I made a silly mistake. Will see if I can fix it up.

nik9000 added a commit that referenced this issue May 10, 2021
@nik9000
Fix bad test
26de09d 
I added a branch in the test for #72277 to fail if I hit an odd case
while debugging. I then wrote code to handle that case and promptly
forgot about it. But I left the odd case detection branch in! And it's
failing the tests. Sorry! This zaps it. It was just a debugging
leftover.

Closes #72883
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nik9000 nik9000

Labels
Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nik9000 @elasticmachine @przemekwitek @danhermann