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.

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

[CI] ShardPathTests testShardPathSelection failing #79249

Closed
markharwood opened this issue Oct 15, 2021 · 3 comments · Fixed by #79393
Closed

[CI] ShardPathTests testShardPathSelection failing #79249

markharwood opened this issue Oct 15, 2021 · 3 comments · Fixed by #79393
Assignees
Labels
Team:Core/Infra Meta label for core/infra team >test-failure Triaged test failures from CI

Comments

@markharwood
Copy link
Contributor

First time seen this and it doesn't reproduce locally.
New test @grcevski ?

Build scan:
https://gradle-enterprise.elastic.co/s/gngncnf4clerq/tests/:server:test/org.elasticsearch.index.shard.ShardPathTests/testShardPathSelection

Reproduction line:
./gradlew ':server:test' --tests "org.elasticsearch.index.shard.ShardPathTests.testShardPathSelection" -Dtests.seed=B20B43CD7F805F0C -Dtests.locale=ru-RU -Dtests.timezone=America/Aruba -Druntime.java=8

Applicable branches:
7.x

Reproduces locally?:
No

Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.index.shard.ShardPathTests&tests.test=testShardPathSelection

Failure excerpt:

java.lang.AssertionError: expected:</var/lib/jenkins/workspace/elastic+elasticsearch+7.x+multijob+platform-support-unix/os/ubuntu-16.04&&immutable/server/build/testrun/test/temp/org.elasticsearch.index.shard.ShardPathTests_B20B43CD7F805F0C-001/tempDir-038/nodes/0/indices/0xDEADBEEF/0> but was:</var/lib/jenkins/workspace/elastic+elasticsearch+7.x+multijob+platform-support-unix/os/ubuntu-16.04&&immutable/server/build/testrun/test/temp/org.elasticsearch.index.shard.ShardPathTests_B20B43CD7F805F0C-001/tempDir-039/nodes/0/indices/0xDEADBEEF/0>

  at __randomizedtesting.SeedInfo.seed([B20B43CD7F805F0C:EA4AFA8B58878A71]: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.index.shard.ShardPathTests.testShardPathSelection(ShardPathTests.java:202)
  at sun.reflect.NativeMethodAccessorImpl.invoke0(NativeMethodAccessorImpl.java:-2)
  at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  at java.lang.reflect.Method.invoke(Method.java:498)
  at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1750)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:938)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:974)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:988)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at org.apache.lucene.util.TestRuleSetupTeardownChained$1.evaluate(TestRuleSetupTeardownChained.java:49)
  at org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:45)
  at org.apache.lucene.util.TestRuleThreadAndTestName$1.evaluate(TestRuleThreadAndTestName.java:48)
  at org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:64)
  at org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:47)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:368)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl.forkTimeoutingTask(ThreadLeakControl.java:817)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl$3.evaluate(ThreadLeakControl.java:468)
  at com.carrotsearch.randomizedtesting.RandomizedRunner.runSingleTest(RandomizedRunner.java:947)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$5.evaluate(RandomizedRunner.java:832)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$6.evaluate(RandomizedRunner.java:883)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$7.evaluate(RandomizedRunner.java:894)
  at org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:45)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at org.apache.lucene.util.TestRuleStoreClassName$1.evaluate(TestRuleStoreClassName.java:41)
  at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
  at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at org.apache.lucene.util.TestRuleAssertionsRequired$1.evaluate(TestRuleAssertionsRequired.java:53)
  at org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:47)
  at org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:64)
  at org.apache.lucene.util.TestRuleIgnoreTestSuites$1.evaluate(TestRuleIgnoreTestSuites.java:54)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:368)
  at java.lang.Thread.run(Thread.java:748)

@markharwood markharwood added Team:Core/Infra Meta label for core/infra team >test-failure Triaged test failures from CI labels Oct 15, 2021
@elasticmachine
Copy link
Collaborator

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

@grcevski
Copy link
Contributor

Yeah I missed this, when I back ported the test from 8.0 I didn't realize ESTestCase newNodeEnvironment adds multiple paths and the assert I added was essentially wrong. I fixed the existing assert when I hit it myself on another backport a2abb99, but I need to fix 8.0 in case we enable multiple paths in the test there too.

grcevski added a commit that referenced this issue Oct 18, 2021
An assert in newly added test didn't anticipate the possibility of
multiple data paths. This change allows any of the shard paths
to be a data path.

Closes #79249
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Core/Infra Meta label for core/infra team >test-failure Triaged test failures from CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants