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] GraphExploreResponseTests.testFromXContent fails #33086

Closed
vladimirdolzhenko opened this issue Aug 23, 2018 · 3 comments
Closed

[CI] GraphExploreResponseTests.testFromXContent fails #33086

vladimirdolzhenko opened this issue Aug 23, 2018 · 3 comments
Assignees
@markharwood
Labels
>test-failure Triaged test failures from CI

Comments

@vladimirdolzhenko
Copy link
Contributor 

Suite: org.elasticsearch.protocol.xpack.graph.GraphExploreResponseTests
  1> [2018-08-23T05:01:44,073][INFO ][o.e.p.x.g.GraphExploreResponseTests] [testFromXContentWithFailures] before test
  1> [2018-08-23T05:01:45,305][INFO ][o.e.p.x.g.GraphExploreResponseTests] [testFromXContentWithFailures] after test
  1> [2018-08-23T05:01:45,310][INFO ][o.e.p.x.g.GraphExploreResponseTests] [testFromXContent] before test
  1> [2018-08-23T05:01:45,525][INFO ][o.e.p.x.g.GraphExploreResponseTests] [testFromXContent] after test
  2> REPRODUCE WITH: ./gradlew :x-pack:protocol:test -Dtests.seed=280221F02AE6E6DA -Dtests.class=org.elasticsearch.protocol.xpack.graph.GraphExploreResponseTests -Dtests.method="testFromXContent" -Dtests.security.manager=true -Dtests.locale=fr-BE -Dtests.timezone=America/Rio_Branco -Dcompiler.java=10 -Druntime.java=8
FAILURE 0.24s J2 | GraphExploreResponseTests.testFromXContent <<< FAILURES!
   > Throwable #1: arrays first differed at element [29]; expected:<org.elasticsearch.protocol.xpack.graph.Connection@75d26158> but was:<org.elasticsearch.protocol.xpack.graph.Connection@5a3a5b58>
   > 	at __randomizedtesting.SeedInfo.seed([280221F02AE6E6DA:96A14E8A3410600]:0)
   > 	at org.elasticsearch.protocol.xpack.graph.GraphExploreResponseTests.assertEqualInstances(GraphExploreResponseTests.java:102)
  2> NOTE: leaving temporary files on disk at: /var/lib/jenkins/workspace/elastic+elasticsearch+6.x+periodic/x-pack/protocol/build/testrun/test/J2/temp/org.elasticsearch.protocol.xpack.graph.GraphExploreResponseTests_280221F02AE6E6DA-001
  2> NOTE: test params are: codec=Asserting(Lucene70): {}, docValues:{}, maxPointsInLeafNode=1761, maxMBSortInHeap=7.7452117757666, sim=RandomSimilarity(queryNorm=false): {}, locale=fr-BE, timezone=America/Rio_Branco
  2> NOTE: Linux 3.10.0-862.3.2.el7.x86_64 amd64/Oracle Corporation 1.8.0_181 (64-bit)/cpus=16,threads=1,free=418005096,total=514850816
  2> NOTE: All tests run in this JVM: [OpenJobResponseTests, InfluencerTests, CategorizationAnalyzerConfigTests, PutWatchResponseTests, BucketInfluencerTests, GraphExploreResponseTests]
   > 	at org.elasticsearch.protocol.xpack.graph.GraphExploreResponseTests.assertEqualInstances(GraphExploreResponseTests.java:37)
   > 	at org.elasticsearch.test.AbstractXContentTestCase.testFromXContent(AbstractXContentTestCase.java:61)
   > 	at org.elasticsearch.test.AbstractXContentTestCase.testFromXContent(AbstractXContentTestCase.java:76)
   > 	at java.lang.Thread.run(Thread.java:748)
   > Caused by: java.lang.AssertionError: expected:<org.elasticsearch.protocol.xpack.graph.Connection@75d26158> but was:<org.elasticsearch.protocol.xpack.graph.Connection@5a3a5b58>
   > 	... 44 more
Completed [23/47] on J2 in 1.51s, 2 tests, 1 failure <<< FAILURES!

logs: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+periodic/2602/console

the result of #33025

it is fully reproducible locally in 6.x branch with

./gradlew :x-pack:protocol:test -Dtests.seed=280221F02AE6E6DA -Dtests.class=org.elasticsearch.protocol.xpack.graph.GraphExploreResponseTests -Dtests.method="testFromXContent" -Dtests.security.manager=true -Dtests.locale=fr-BE -Dtests.timezone=America/Rio_Branco -Dcompiler.java=10 -Druntime.java=8
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@markharwood markharwood self-assigned this Aug 23, 2018
markharwood added a commit that referenced this issue Aug 23, 2018
@markharwood
Test fix - GraphExploreResponseTests should not randomise array elements
6a92a35 
Closes #33086
@markharwood
Copy link
Contributor

Fixed in 6.x 6a92a35 and master f860e58

@vladimirdolzhenko
Copy link
Contributor Author

thanks @markharwood

martijnvg added a commit that referenced this issue Aug 24, 2018
@martijnvg
Merge remote-tracking branch 'es/master' into ccr
82592dd 
* es/master: (62 commits)
  [DOCS] Add docs for Application Privileges (#32635)
  Add versions 5.6.12 and 6.4.1
  Do NOT allow termvectors on nested fields (#32728)
  [Rollup] Return empty response when aggs are missing (#32796)
  [TEST] Add some ACL yaml tests for Rollup (#33035)
  Move non duplicated actions back into xpack core (#32952)
  Test fix - GraphExploreResponseTests should not randomise array elements Closes #33086
  Use `addIfAbsent` instead of checking if an element is contained
  TESTS: Fix Random Fail in MockTcpTransportTests (#33061)
  HLRC: Fix Compile Error From Missing Throws (#33083)
  [DOCS] Remove reload password from docs cf. #32889
  HLRC: Add ML Get Buckets API (#33056)
  Watcher: Improve error messages for CronEvalTool (#32800)
  Search: Support of wildcard on docvalue_fields (#32980)
  Change query field expansion (#33020)
  INGEST: Cleanup Redundant Put Method (#33034)
  SQL: skip uppercasing/lowercasing function tests for AZ locales as well (#32910)
  Fix the default pom file name (#33063)
  Switch ml basic tests to new style Requests (#32483)
  Switch some watcher tests to new style Requests (#33044)
  ...
martijnvg added a commit that referenced this issue Aug 24, 2018
@martijnvg
Merge remote-tracking branch 'es/6.x' into ccr-6.x
943caaf 
* es/6.x: (58 commits)
  [DOCS] Add docs for Application Privileges (#32635)
  Add versions 5.6.12 and 6.4.1
  [Rollup] Return empty response when aggs are missing (#32796)
  [TEST] Add some ACL yaml tests for Rollup (#33035)
  Test fix - GraphExploreResponseTests should not randomise array elements Closes #33086
  Use `addIfAbsent` instead of checking if an element is contained
  HLRC: Fix Compile Error From Missing Throws (#33083)
  [DOCS] Remove reload password from docs cf. #32889
  Use a dedicated ConnectionManger for RemoteClusterConnection (#32988)
  Watcher: Improve error messages for CronEvalTool (#32800)
  HLRC: Add ML Get Buckets API (#33056)
  Change query field expansion (#33020)
  Search: Support of wildcard on docvalue_fields (#32980)
  Add beta label to MSI on install Elasticsearch page (#28126)
  SQL: skip uppercasing/lowercasing function tests for AZ locales as well (#32910)
  [DOCS] Drafts Elasticsearch 6.4.0 release notes (#33039)
  Fix the default pom file name (#33063)
  Fix backport of switch ml basic tests to new style Requests (#32483)
  Switch ml basic tests to new style Requests (#32483)
  Switch some watcher tests to new style Requests (#33044)
  ...
jasontedor added a commit to jasontedor/elasticsearch that referenced this issue Aug 24, 2018
@jasontedor
Merge branch 'ccr' into ccr-fetch-exceptions
3efdf20 
* ccr: (71 commits)
  Make CCR QA tests build again (elastic#33113)
  Add hook to skip asserting x-content equivalence (elastic#33114)
  Muted testListenersThrowingExceptionsDoNotCauseOtherListenersToBeSkipped
  [Rollup] Move getMetadata() methods out of rollup config objects (elastic#32579)
  fixed not returning response instance
  Muted testEmptyAuthorizedIndicesSearchForAllDisallowNoIndices
  Update Google Cloud Storage Library for Java (elastic#32940)
  Remove unsupported Version.V_5_* (elastic#32937)
  Required changes after merging in master branch.
  [DOCS] Add docs for Application Privileges (elastic#32635)
  Add versions 5.6.12 and 6.4.1
  Do NOT allow termvectors on nested fields (elastic#32728)
  [Rollup] Return empty response when aggs are missing (elastic#32796)
  [TEST] Add some ACL yaml tests for Rollup (elastic#33035)
  Move non duplicated actions back into xpack core (elastic#32952)
  Test fix - GraphExploreResponseTests should not randomise array elements Closes elastic#33086
  Use `addIfAbsent` instead of checking if an element is contained
  TESTS: Fix Random Fail in MockTcpTransportTests (elastic#33061)
  HLRC: Fix Compile Error From Missing Throws (elastic#33083)
  [DOCS] Remove reload password from docs cf. elastic#32889
  ...
@cbuescher cbuescher mentioned this issue Aug 29, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markharwood markharwood

Labels
>test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@markharwood @vladimirdolzhenko @elasticmachine