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

Cherry-pick #16139 to 7.x: Regenerate expected files after changes in date parsing #16175

Closed
wants to merge 5 commits into from

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Feb 6, 2020

Cherry-pick of PR #16139 to 7.x branch. Original message:

Elasticsearch has modified the behaviour on date parsing when the date
doesn't include timezone data. Regenerate a couple of golden files that
are affected by this change.

New values are actually correct, so this will fix some issues on logs without
timezone information.

I think the change in Elasticsearch is elastic/elasticsearch#51215

We will need to backport this for 7.7 as this change is going to be included
in Elasticsearch 7.7.

Elasticsearch has modified the behaviour on date parsing when the date
doesn't include timezone data. Regenerate a couple of golden files that
are affected by this change.

(cherry picked from commit e61c4c4)
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's wait for CI to pass first 👍 thanks for fixing this.

@jsoriano jsoriano self-assigned this Feb 6, 2020
@graphaelli
Copy link
Member

Looks like this is needed now, can we get this branch up to date with 7.x please?

@jsoriano
Copy link
Member Author

jsoriano commented Mar 3, 2020

@graphaelli branch updated, but feel free to bring this change to #16699 and merge it all together.

@kaiyan-sheng
Copy link
Contributor

ci failure in libbeat is caused by #16764

@graphaelli
Copy link
Member

merged as part of #16699, thanks

@graphaelli graphaelli closed this Mar 5, 2020
@jsoriano jsoriano deleted the backport_16139_7.x branch March 5, 2020 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants