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.

Sign up for GitHub

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

Jump to bottom

Cherry-pick #15935 to 7.x: [Elastic Log Driver] Fix file check bug, deal with logging #16094

Merged
merged 2 commits into from
Feb 5, 2020
Merged

Cherry-pick #15935 to 7.x: [Elastic Log Driver] Fix file check bug, deal with logging #16094

merged 2 commits into from
Feb 5, 2020

Conversation

fearful-symmetry
Copy link
Contributor

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

This addresses a bug where we weren't properly checking to see if a meta.json file existed, and also a default debug log level. Users can now config the log level with LOG_DRIVER_LEVEL

@fearful-symmetry
[Elastic Log Driver] Fix file check bug, deal with logging (elastic#1…
56bc580 
…5935)

* fix file check bug, deal with logging

* add logging env fix

* remove space

* fix nil pointer deref

* add missing blocking read call

(cherry picked from commit d60e65b)
@fearful-symmetry fearful-symmetry requested a review from a team February 5, 2020 00:03
@fearful-symmetry fearful-symmetry merged commit aa25d3c into elastic:7.x Feb 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaiyan-sheng kaiyan-sheng kaiyan-sheng approved these changes

Assignees
No one assigned
Labels
backport bug review v7.6.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fearful-symmetry @kaiyan-sheng