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] DynamicMappingIT testMappingsPropagatedToMasterNodeImmediately fails #37816

Closed
droberts195 opened this issue Jan 24, 2019 · 8 comments
Closed
Assignees
Labels
:Search Foundations/Mapping Index mappings, including merging and defining field types Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test-failure Triaged test failures from CI

Comments

@droberts195
Copy link
Contributor

DynamicMappingIT testMappingsPropagatedToMasterNodeImmediately failed in https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+intake/1152/console

The error is:

12:34:53 FAILURE 0.22s J6 | DynamicMappingIT.testMappingsPropagatedToMasterNodeImmediately <<< FAILURES!
12:34:53    > Throwable #1: java.lang.AssertionError
12:34:53    > 	at __randomizedtesting.SeedInfo.seed([C13A3C57FAFE9674:4CE8785023961AD4]:0)
12:34:53    > 	at org.elasticsearch.index.mapper.DynamicMappingIT.assertMappingsHaveField(DynamicMappingIT.java:61)
12:34:53    > 	at org.elasticsearch.index.mapper.DynamicMappingIT.testMappingsPropagatedToMasterNodeImmediately(DynamicMappingIT.java:75)
12:34:53    > 	at java.lang.Thread.run(Thread.java:748)

I could not reproduce this locally using:

./gradlew :server:integTest \
  -Dtests.seed=C13A3C57FAFE9674 \
  -Dtests.class=org.elasticsearch.index.mapper.DynamicMappingIT \
  -Dtests.method="testMappingsPropagatedToMasterNodeImmediately" \
  -Dtests.security.manager=true \
  -Dtests.locale=cs-CZ \
  -Dtests.timezone=Europe/Monaco \
  -Dcompiler.java=11 \
  -Druntime.java=8
@droberts195 droberts195 added :Search Foundations/Mapping Index mappings, including merging and defining field types >test-failure Triaged test failures from CI labels Jan 24, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@droberts195
Copy link
Contributor Author

I muted this test for 6.x in 1bcc7b1

@droberts195
Copy link
Contributor Author

This also affects master.

This failure occurred in https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+intake/1493/console:

13:18:38 FAILURE 0.61s J4 | DynamicMappingIT.testMappingsPropagatedToMasterNodeImmediately <<< FAILURES!
13:18:38    > Throwable #1: java.lang.AssertionError
13:18:38    > 	at __randomizedtesting.SeedInfo.seed([A491240509B193BC:29436002D0D91F1C]:0)
13:18:38    > 	at org.elasticsearch.index.mapper.DynamicMappingIT.assertMappingsHaveField(DynamicMappingIT.java:70)
13:18:38    > 	at org.elasticsearch.index.mapper.DynamicMappingIT.testMappingsPropagatedToMasterNodeImmediately(DynamicMappingIT.java:84)
13:18:38    > 	at java.lang.Thread.run(Thread.java:748)

The repro command for this build was:

./gradlew :server:integTest \
  -Dtests.seed=A491240509B193BC \
  -Dtests.class=org.elasticsearch.index.mapper.DynamicMappingIT \
  -Dtests.method="testMappingsPropagatedToMasterNodeImmediately" \
  -Dtests.security.manager=true \
  -Dtests.locale=pt-PT \
  -Dtests.timezone=America/Eirunepe \
  -Dcompiler.java=11 \
  -Druntime.java=8

I muted the test for master too in a81931b

@droberts195 droberts195 changed the title [CI] DynamicMappingIT testMappingsPropagatedToMasterNodeImmediately fails in 6.x [CI] DynamicMappingIT testMappingsPropagatedToMasterNodeImmediately fails Jan 24, 2019
@alpar-t
Copy link
Contributor

alpar-t commented Jan 24, 2019

This reproduced locally for me with a loop:

while ./gradlew :server:integTest -Dtests.seed=6F0955888A93B810 -Dtests.class=org.elasticsearch.index.mapper.DynamicMappingIT  ; do 
        echo  AGAIN
done

@cbuescher
Copy link
Member

This reproduced locally for me with a loop:

Which branch is this? Asking because the seed which triggerst the behaviour might be different on different branches.

@cbuescher
Copy link
Member

I get it to fail running in a loop on master with seed A491240509B193BC:29436002D0D91F1C maybe 2 out of ten times.

@cbuescher
Copy link
Member

btw. this looks very much related to #37817 and vice versa, maybe we can close one of those two issues.

@dnhatn dnhatn self-assigned this Jan 25, 2019
@dnhatn dnhatn closed this as completed in 3ccd488 Jan 25, 2019
@dnhatn
Copy link
Member

dnhatn commented Jan 25, 2019

This test is obsolete since #31140 where an index request with dynamic mapping update no longer requires acking. I pushed 3ccd488 to this test. /cc @ywelsch.

dnhatn added a commit that referenced this issue Jan 25, 2019
This test is obsolete since #31140 where an index request with dynamic
mapping update no longer requires acking.

Closes #37816
@javanna javanna added the Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch label Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Search Foundations/Mapping Index mappings, including merging and defining field types Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test-failure Triaged test failures from CI
Projects
None yet
Development

No branches or pull requests

6 participants