Skip to content

[CI] RemoteRequestBuildersTests testInitialSearchParamsFields failing #74131

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
Closed
not-napoleon opened this issue Jun 15, 2021 · 1 comment · Fixed by #74914
Closed

[CI] RemoteRequestBuildersTests testInitialSearchParamsFields failing #74131

not-napoleon opened this issue Jun 15, 2021 · 1 comment · Fixed by #74914
Assignees
@henningandersen
Labels
:Distributed Indexing/Reindex Issues relating to reindex that are not caused by issues further down Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. >test-failure Triaged test failures from CI

Comments

@not-napoleon
Copy link
Member

Build scan:
https://gradle-enterprise.elastic.co/s/cvav7jlqabw74/tests/:modules:reindex:test/org.elasticsearch.index.reindex.remote.RemoteRequestBuildersTests/testInitialSearchParamsFields

Reproduction line:
./gradlew ':modules:reindex:test' --tests "org.elasticsearch.index.reindex.remote.RemoteRequestBuildersTests.testInitialSearchParamsFields" -Dtests.seed=B911D2E4671CC726 -Dtests.locale=ar-BH -Dtests.timezone=Australia/Lord_Howe -Druntime.java=8

Applicable branches:
7.x

Reproduces locally?:
Yes

Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.index.reindex.remote.RemoteRequestBuildersTests&tests.test=testInitialSearchParamsFields

Failure excerpt:

java.lang.AssertionError: 
Expected: map containing ["stored_fields"->"_source,_id"]
     but: map was [<size=-1>, <fields=_source,_id>, <version=false>]

  at __randomizedtesting.SeedInfo.seed([B911D2E4671CC726:DE23B75A43C9A8ED]:0)
  at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
  at org.junit.Assert.assertThat(Assert.java:956)
  at org.junit.Assert.assertThat(Assert.java:923)
  at org.elasticsearch.index.reindex.remote.RemoteRequestBuildersTests.testInitialSearchParamsFields(RemoteRequestBuildersTests.java:141)
  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)
@not-napoleon not-napoleon added :Distributed Indexing/Reindex Issues relating to reindex that are not caused by issues further down >test-failure Triaged test failures from CI labels Jun 15, 2021
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Jun 15, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@henningandersen henningandersen self-assigned this Jun 16, 2021
henningandersen added a commit to henningandersen/elasticsearch that referenced this issue Jul 5, 2021
@henningandersen
Remote reindex stored fields since 5.0.0
dd7be00 
Test would assume stored fields were available since 5.0.0.alpha4,
while production code only enables them after 5.0.0 (final, aka 99).

Closes elastic#74131
@henningandersen henningandersen mentioned this issue Jul 5, 2021
Merged
henningandersen added a commit that referenced this issue Aug 4, 2021
@henningandersen
Remote reindex stored fields since 5.0.0 (#74914)
b19f601 
Test would assume stored fields were available since 5.0.0.alpha4,
while production code only enables them after 5.0.0 (final, aka 99).

Closes #74131
henningandersen added a commit that referenced this issue Aug 4, 2021
@henningandersen
Remote reindex stored fields since 5.0.0 (#74914)
18430f3 
Test would assume stored fields were available since 5.0.0.alpha4,
while production code only enables them after 5.0.0 (final, aka 99).

Closes #74131
henningandersen added a commit that referenced this issue Aug 4, 2021
@henningandersen
Remote reindex stored fields since 5.0.0 (#74914)
e225023 
Test would assume stored fields were available since 5.0.0.alpha4,
while production code only enables them after 5.0.0 (final, aka 99).

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

@henningandersen henningandersen

Labels
:Distributed Indexing/Reindex Issues relating to reindex that are not caused by issues further down Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remote reindex stored fields since 5.0.0 henningandersen/elasticsearch
3 participants
@not-napoleon @elasticmachine @henningandersen