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

TransportAnalyticsStatsActionTests fail on master #54744

Closed
mayya-sharipova opened this issue Apr 3, 2020 · 2 comments · Fixed by #54753
Closed

TransportAnalyticsStatsActionTests fail on master #54744

mayya-sharipova opened this issue Apr 3, 2020 · 2 comments · Fixed by #54753
Assignees
Labels
:Analytics/Aggregations Aggregations >test-failure Triaged test failures from CI

Comments

@mayya-sharipova
Copy link
Contributor

./gradlew ':x-pack:plugin:analytics:test' --tests "org.elasticsearch.xpack.analytics.action.TransportAnalyticsStatsActionTests.test"
-Dtests.seed=C06A02480587257E
-Dtests.security.manager=true
-Dtests.locale=es-PY
-Dtests.timezone=Africa/Luanda
-Dcompiler.java=13

Test with this seed reproducibly fails for me.

As the test overall doesn't fail, only with this particular seed, I am not going to mute it for now.

Log: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+multijob-unix-compatibility/os=sles-12&&immutable/673/console
Build Scans: https://gradle-enterprise.elastic.co/s/nmicdnymlrs4a

Console output:

java.lang.AssertionError: 
Expected: <0>
     but: was null
	at __randomizedtesting.SeedInfo.seed([C06A02480587257E:483E3D92AB7B4886]:0)
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
	at org.junit.Assert.assertThat(Assert.java:956)
	at org.junit.Assert.assertThat(Assert.java:923)
	at org.elasticsearch.xpack.analytics.action.TransportAnalyticsStatsActionTests.test(TransportAnalyticsStatsActionTests.java:66)
@mayya-sharipova mayya-sharipova added >test-failure Triaged test failures from CI :Analytics/Aggregations Aggregations labels Apr 3, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (:Analytics/Aggregations)

mayya-sharipova added a commit that referenced this issue Apr 3, 2020
@mayya-sharipova
Copy link
Contributor Author

Actually, there were quite a number of failures so the test was muted.

@nik9000 nik9000 assigned nik9000 and imotov and unassigned nik9000 Apr 3, 2020
imotov added a commit to imotov/elasticsearch that referenced this issue Apr 3, 2020
Adds missing t_test usage stats to XContent output.

Fixes elastic#54744
imotov added a commit that referenced this issue Apr 3, 2020
Adds missing t_test usage stats to XContent output.

Fixes #54744
imotov added a commit to imotov/elasticsearch that referenced this issue Apr 6, 2020
Adds missing t_test usage stats to XContent output.

Fixes elastic#54744
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations >test-failure Triaged test failures from CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants