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] SearchHitTests.testSerializeShardTarget fails on 6.x #37214

Closed
tlrx opened this issue Jan 8, 2019 · 3 comments
Closed

[CI] SearchHitTests.testSerializeShardTarget fails on 6.x #37214

tlrx opened this issue Jan 8, 2019 · 3 comments
Assignees
@javanna
Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI v6.7.0

Comments

@tlrx
Copy link
Member

tlrx commented Jan 8, 2019

After #36997, the SearchHitTests.testSerializeShardTarget needs to be adapted to work on 6.x when a random version < 5.6 is picked up and the cluster alias is not deserialized.

See:
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+release-tests/317/consoleFull

I can reproduce on 6.x with:

./gradlew :server:unitTest -Dtests.seed=6E493E03085BDF45 -Dtests.class=org.elasticsearch.search.SearchHitTests -Dtests.method="testSerializeShardTarget" -Dtests.security.manager=true -Dbuild.snapshot=true -Dtests.jvm.argline="-Dbuild.snapshot=true" -Dtests.locale=sr-BA -Dtests.timezone=Australia/ACT -Dcompiler.java=11 -Druntime.java=8
@tlrx tlrx added :Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI v6.7.0 labels Jan 8, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

tlrx added a commit that referenced this issue Jan 8, 2019
@tlrx
[Tests] Mute SearchHitTests.testSerializeShardTarget
3bca3d4 
Reported in #37214
@tlrx
Copy link
Member Author

tlrx commented Jan 8, 2019

Test muted on 6.x in 3bca3d4

@javanna javanna self-assigned this Jan 8, 2019
javanna added a commit that referenced this issue Jan 8, 2019
@javanna
[TEST] Adjust version in SearchHitTests
890fa34 
Closes #37214
@javanna
Copy link
Member

javanna commented Jan 8, 2019

Fixed in 890fa34

@javanna javanna closed this as completed Jan 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@javanna javanna

Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI v6.7.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tlrx @javanna @elasticmachine