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

Tcp and nio transport test failures on version bump to 7.0 #25893

Closed
rjernst opened this issue Jul 25, 2017 · 1 comment
Closed

Tcp and nio transport test failures on version bump to 7.0 #25893

rjernst opened this issue Jul 25, 2017 · 1 comment
Assignees
@s1monw
Labels
>test Issues or PRs that are addressing/adding tests v7.0.0-beta1

Comments

@rjernst
Copy link
Member

rjernst commented Jul 25, 2017

Two test failures, but they are the exact same:

  2> REPRODUCE WITH: gradle :test:framework:test -Dtests.seed=15A33AAAA8383DC4 -Dtests.class=org.elasticsearch.transport.nio.SimpleNioTransportTests -Dtests.method="testTransportStatsWithException" -Dtests.security.manager=true -Dtests.locale=hr -Dtests.timezone=Asia/Nicosia
FAILURE 0.06s J0 | SimpleNioTransportTests.testTransportStatsWithException <<< FAILURES!
   > Throwable #1: java.lang.AssertionError: expected:<189> but was:<187>
   >    at __randomizedtesting.SeedInfo.seed([15A33AAAA8383DC4:D9231B7D2322A8CF]:0)
   >    at org.elasticsearch.transport.AbstractSimpleTransportTestCase.testTransportStatsWithException(AbstractSimpleTransportTestCase.java:2425)
   >    at java.lang.Thread.run(Thread.java:745)

  2> REPRODUCE WITH: gradle :test:framework:test -Dtests.seed=15A33AAAA8383DC4 -Dtests.class=org.elasticsearch.transport.MockTcpTransportTests -Dtests.method="testTransportStatsWithException" -Dtests.security.manager=true -Dtests.locale=lv-LV -Dtests.timezone=America/Maceio
FAILURE 0.02s J3 | MockTcpTransportTests.testTransportStatsWithException <<< FAILURES!
   > Throwable #1: java.lang.AssertionError: expected:<189> but was:<187>
   >    at __randomizedtesting.SeedInfo.seed([15A33AAAA8383DC4:D9231B7D2322A8CF]:0)
   >    at org.elasticsearch.transport.AbstractSimpleTransportTestCase.testTransportStatsWithException(AbstractSimpleTransportTestCase.java:2425)
   >    at java.lang.Thread.run(Thread.java:745)
@rjernst rjernst added v7.0.0 >test Issues or PRs that are addressing/adding tests labels Jul 25, 2017
@s1monw s1monw self-assigned this Jul 28, 2017
s1monw added a commit to rjernst/elasticsearch that referenced this issue Jul 28, 2017
@s1monw
Fix expected bytes in tests due to new major version
d6e2c06 
In 6.x we had to serialize the 2 byte marker for the transport address type
for BWC. Now in 6.x this doesn't exist and we write 2 bytes less when we serialize
a transport address to a min compat node.

Closes elastic#25893
@s1monw
Copy link
Contributor

s1monw commented Jul 28, 2017

this will be fixed in #25876

@s1monw s1monw closed this as completed Jul 28, 2017
rjernst added a commit that referenced this issue Aug 1, 2017
@rjernst
Update version to 7.0.0-alpha1 (#25876)
072281d 
This commit updates the version for master to 7.0.0-alpha1. It also adds
the 6.1 version constant, and fixes many tests, as well as marking some
as awaits fix.

Closes #25893
Closes #25870
@jimczi jimczi added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@s1monw s1monw

Labels
>test Issues or PRs that are addressing/adding tests v7.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rjernst @s1monw @jimczi