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 broken becasue filebeat haproxy, redis and system test failed with wrong timestamp #9849

Closed
kaiyan-sheng opened this issue Jan 2, 2019 · 1 comment
Assignees
Labels
Filebeat Filebeat Team:Integrations Label for the Integrations team

Comments

@kaiyan-sheng
Copy link
Contributor

Redis filebeat module: 98738:M 30 May 12:23:52.442 * Saving the final RDB snapshot before exiting. is the log entry in filebeat/module/redis/log/test/test.log but it doesn’t have a year on the timestamp. So in test.log-expected.json, the timestamp used to be 2018 but now it changed to 2019. That's why CI is failing with error like: https://travis-ci.org/elastic/beats/jobs/474342333#L2082

Same problem in haproxy filebeat module: Sep 20 15:42:59 1.2.3.4 haproxy[24551]: Connect from 1.2.3.4:40780 to 1.2.3.4:5000 (main/HTTP) is from filebeat/module/haproxy/log/test/default.log. CI failure example: https://travis-ci.org/elastic/beats/jobs/474342333#L1373

These errors are also exist in master branch, for example: https://travis-ci.org/elastic/beats/jobs/474520532#L1344

@kaiyan-sheng kaiyan-sheng added Team:Integrations Label for the Integrations team Filebeat Filebeat labels Jan 2, 2019
@kaiyan-sheng kaiyan-sheng self-assigned this Jan 2, 2019
@kaiyan-sheng kaiyan-sheng changed the title CI broken becasue filebeat haproxy and redis test_modules.py failed with wrong timestamp CI broken becasue filebeat haproxy, redis and system test failed with wrong timestamp Jan 2, 2019
@andrewkroh
Copy link
Member

@jsoriano's comment is relevant here: #9840 (comment)

To avoid this happening on next year, should we add exceptions for these timestamps as the ones we have for icinga? Or replace the years before checking?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Filebeat Filebeat Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants