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

ESJsonLayoutTests fail on string comparison assertion #44565

Closed
przemekwitek opened this issue Jul 18, 2019 · 5 comments
Closed

ESJsonLayoutTests fail on string comparison assertion #44565

przemekwitek opened this issue Jul 18, 2019 · 5 comments
Labels
:Core/Infra/Core Core issues without another label >test-failure Triaged test failures from CI

Comments

@przemekwitek
Copy link
Contributor

FAILURE #7.x a5ad594 multijob-windows-compatibility/os=windows-2012-r2 - 20190718105648-DFE773DC

Recently introduced test (ESJsonLayoutTests) fails on Windows. Most likely this is due to new line characters being different:

14:40:38     java.lang.AssertionError: 
14:40:38     Expected: "{\"type\": \"server\", \"timestamp\": \"%d{yyyy-MM-dd'T'HH:mm:ss,SSSZZ}\", \"level\": \"%p\", \"component\": \"%c{1.}\", \"cluster.name\": \"${sys:es.logs.cluster_name}\", \"node.name\": \"%node_name\", \"message\": \"%notEmpty{%enc{%marker}{JSON} }%enc{%.-10000m}{JSON}\"%notEmpty{, %node_and_cluster_id }%exceptionAsJson }\n"
14:40:38          but: was "{\"type\": \"server\", \"timestamp\": \"%d{yyyy-MM-dd'T'HH:mm:ss,SSSZZ}\", \"level\": \"%p\", \"component\": \"%c{1.}\", \"cluster.name\": \"${sys:es.logs.cluster_name}\", \"node.name\": \"%node_name\", \"message\": \"%notEmpty{%enc{%marker}{JSON} }%enc{%.-10000m}{JSON}\"%notEmpty{, %node_and_cluster_id }%exceptionAsJson }\r\n"

Log: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+7.x+multijob-windows-compatibility/os=windows-2012-r2/15/console
Build Scans: https://gradle.com/s/rigslz4ooby6m, https://gradle.com/s/tuogchgpwby4a, https://gradle.com/s/au527lzzgydvq

@przemekwitek przemekwitek added the >test-failure Triaged test failures from CI label Jul 18, 2019
@przemekwitek przemekwitek added the :Core/Infra/Core Core issues without another label label Jul 18, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@alpar-t
Copy link
Contributor

alpar-t commented Jul 18, 2019

This fails on master as well , I have an un-merged fix in b72be5c

@pgomulka pgomulka removed their assignment Jul 18, 2019
@andrershov
Copy link
Contributor

@pgomulka pgomulka self-assigned this Jul 24, 2019
@pgomulka
Copy link
Contributor

fixed by #44675

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >test-failure Triaged test failures from CI
Projects
None yet
Development

No branches or pull requests

6 participants