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] HistoryIntegrationTests.testThatHistoryContainsStatus failures #38693

Closed
tvernum opened this issue Feb 11, 2019 · 4 comments
Closed

[CI] HistoryIntegrationTests.testThatHistoryContainsStatus failures #38693

tvernum opened this issue Feb 11, 2019 · 4 comments
Labels
:Data Management/Watcher >test-failure Triaged test failures from CI

Comments

@tvernum
Copy link
Contributor

tvernum commented Feb 11, 2019

Test fails reproducibly.
CI failed on Java 12, but it reproduces on Java 11 for me.

From: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+matrix-java-periodic/ES_BUILD_JAVA=java11,ES_RUNTIME_JAVA=openjdk12,nodes=immutable&&linux&&docker/235/console

05:50:09 FAILURE 1.44s J3 | HistoryIntegrationTests.testThatHistoryContainsStatus <<< FAILURES!
05:50:09    > Throwable #1: java.lang.AssertionError: 
05:50:09    > Expected: is "2019-02-11T05:50:07.998Z"
05:50:09    >      but: was "2019-02-11T05:50:07.998586Z"
05:50:09    > 	at __randomizedtesting.SeedInfo.seed([F30FA9CCD903E4B5:FE0ACADB0B20E6F7]:0)
05:50:09    > 	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)

./gradlew :x-pack:plugin:watcher:unitTest \
  -Dtests.seed=F30FA9CCD903E4B5 \
  -Dtests.class=org.elasticsearch.xpack.watcher.test.integration.HistoryIntegrationTests \
  -Dtests.method="testThatHistoryContainsStatus" \
  -Dtests.security.manager=true \
  -Dtests.locale=it-IT \
  -Dtests.timezone=Asia/Shanghai \
  -Dcompiler.java=11 \
  -Druntime.java=12

Relates: #38505, #27330

@tvernum tvernum added >test-failure Triaged test failures from CI :Data Management/Watcher labels Feb 11, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@pgomulka pgomulka self-assigned this Feb 11, 2019
@pgomulka
Copy link
Contributor

should be fixed by #38626

@pgomulka
Copy link
Contributor

but might as well change this to use a formatter that it uses during toXContent (as per comment #38619 (comment))

XContentElasticsearchExtension.DEFAULT_FORMATTER

pgomulka added a commit that referenced this issue Feb 11, 2019
pgomulka added a commit to pgomulka/elasticsearch that referenced this issue Feb 11, 2019
pgomulka added a commit to pgomulka/elasticsearch that referenced this issue Feb 11, 2019
pgomulka added a commit that referenced this issue Feb 11, 2019
@pgomulka
Copy link
Contributor

pgomulka commented Apr 8, 2019

closing as it is fixed in #38626

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Watcher >test-failure Triaged test failures from CI
Projects
None yet
Development

No branches or pull requests

3 participants