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

Fix filebeat nginx module ingest timezone #13369

Merged
merged 5 commits into from
Sep 4, 2019
Merged

Fix filebeat nginx module ingest timezone #13369

merged 5 commits into from
Sep 4, 2019

Conversation

pragkent
Copy link
Contributor

@pragkent pragkent commented Aug 28, 2019

This pull request fixes timezone parsing for nginx module.

Just like #13308 fixes ingest timezone parsing for system module.

@pragkent pragkent requested a review from a team as a code owner August 28, 2019 09:42
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

1 similar comment
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@jsoriano jsoriano added Team:Integrations Label for the Integrations team Filebeat Filebeat module needs_backport PR is waiting to be backported to other branches. review v7.3.2 v7.4.0 labels Aug 29, 2019
@jsoriano
Copy link
Member

ok to test

@jsoriano
Copy link
Member

jsoriano commented Sep 2, 2019

@pragkent I have added a commit here so errors parsing date in access log are not ignored, as they weren't before, let me know if you think this can be a problem.

@pragkent
Copy link
Contributor Author

pragkent commented Sep 3, 2019

@jsoriano agree, that's great.

@jsoriano
Copy link
Member

jsoriano commented Sep 3, 2019

jenkins, test this again please

@jsoriano jsoriano merged commit 5d92b2b into elastic:master Sep 4, 2019
jsoriano pushed a commit to jsoriano/beats that referenced this pull request Sep 4, 2019
@jsoriano jsoriano removed the needs_backport PR is waiting to be backported to other branches. label Sep 4, 2019
jsoriano pushed a commit to jsoriano/beats that referenced this pull request Sep 4, 2019
jsoriano added a commit that referenced this pull request Sep 5, 2019
(cherry picked from commit 5d92b2b)

Co-authored-by: Kent Wang <pragkent@gmail.com>
jsoriano added a commit that referenced this pull request Sep 5, 2019
(cherry picked from commit 5d92b2b)

Co-authored-by: Kent Wang <pragkent@gmail.com>
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
)

(cherry picked from commit 89290c4)

Co-authored-by: Kent Wang <pragkent@gmail.com>
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
)

(cherry picked from commit 89290c4)

Co-authored-by: Kent Wang <pragkent@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants