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

*: support output logs to multiple files #9579

Merged
merged 12 commits into from
Apr 17, 2018
Merged

*: support output logs to multiple files #9579

merged 12 commits into from
Apr 17, 2018

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Apr 17, 2018

zap supports it, so etcd can use it when --logger zap --log-output a.log,b.log,c.log,stderr.

@gyuho
embed: support multiple log outputs for structured logging
f6f1e0c 
Zap support multi-writer by default.

Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
CHANGELOG-3.4: highlight multi-log writer change
954afc9 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
embed: change "Config.LogOutput" to []string
bf93753 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
etcdmain: accept multiple values in "--log-output"
1974785 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
snapshot: set string for LogOutput
3e99487 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
integration: set strings for LogOutput
cd7bfe1 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
CHANGELOG-3.4: highlight embed.Config.LogOutput field change
8049df1 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
CHANGELOG-3.4: add "LogOutput" change
671e4a5 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
functional: use strings for LogOutput
8ee8778 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho gyuho force-pushed the log-output-multiple-files branch from f28a3bc to 8ee8778 Compare April 17, 2018 20:04
@gyuho gyuho merged commit 9c0c3cd into etcd-io:master Apr 17, 2018
@gyuho gyuho deleted the log-output-multiple-files branch April 17, 2018 21:39
@nolouch nolouch mentioned this pull request Jun 4, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gyuho