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 journald with structured logging zap, rename to "--log-outputs" #9624

Merged
merged 22 commits into from
Apr 25, 2018
Merged

*: support journald with structured logging zap, rename to "--log-outputs" #9624

merged 22 commits into from
Apr 25, 2018

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Apr 24, 2018 •
edited
Loading

Confirmed that logs are redirected to journald when running as a systemd unit.

@gyuho gyuho changed the title *: support journald logging with structured logging zap *: support journald with structured logging zap Apr 25, 2018
gyuho added 11 commits April 25, 2018 10:05
@gyuho
pkg/logutil: add "NewJournaldWriter"
fcbb303 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
pkg/logutil: add "NewGRPCLoggerV2FromZapCore"
1fa80bf 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
embed: set journald logging with zap
48d5542 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
pkg/logutil: add "NewRaftLoggerFromZapCore"
6df3179 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
embed: change "loggerConfig" to "*zap.Config"
69c51e2 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
pkg/logutil: change "NewRaftLogger" function signature
a34dd27 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
embed: add "loggerCore" field for Raft logger
d33a74d 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
etcdserver: add "LoggerCore" field for Raft logger
3ea7a5d 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
embed: rename "LogOutput" to "LogOutputs"
f99cb35 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
CHANGELOG: track logger changes
d4987f4 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
Documentation/upgrades: highlight "embed.Config.LogOutputs" change
53c5cd5 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
functional/rpcpb: change to LogOutputs
a113d6c 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
CHANGELOG: highlight "--log-outputs" flag
0e4c94c 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
Documentation/upgrades: highlight "--log-outputs"
22609a6 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
etcd.conf.yml.sample: rename to "log-outputs"
1539fdd 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
embed,etcdmain: rename to "--log-outputs" flag
af5bc43 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
functional: rename to "log-outputs"
a6c30ea 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
*: rename to "--log-outputs" in docs
58a6034 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho
*: change typo in "auto-compaction-retention" field
b748abc 
Signed-off-by: Gyuho Lee <[email protected]>
@gyuho gyuho changed the title *: support journald with structured logging zap *: support journald with structured logging zap, rename to "--log-outputs" Apr 25, 2018
@gyuho gyuho merged commit ccf959c into etcd-io:master Apr 25, 2018
@gyuho gyuho deleted the journald branch April 25, 2018 21:22
@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