Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 <gyuhox@gmail.com>
@gyuho
pkg/logutil: add "NewGRPCLoggerV2FromZapCore"
1fa80bf 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
embed: set journald logging with zap
48d5542 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
pkg/logutil: add "NewRaftLoggerFromZapCore"
6df3179 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
embed: change "loggerConfig" to "*zap.Config"
69c51e2 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
pkg/logutil: change "NewRaftLogger" function signature
a34dd27 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
embed: add "loggerCore" field for Raft logger
d33a74d 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
etcdserver: add "LoggerCore" field for Raft logger
3ea7a5d 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
embed: rename "LogOutput" to "LogOutputs"
f99cb35 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
CHANGELOG: track logger changes
d4987f4 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
Documentation/upgrades: highlight "embed.Config.LogOutputs" change
53c5cd5 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
gyuho added 8 commits April 25, 2018 11:05
@gyuho
functional/rpcpb: change to LogOutputs
a113d6c 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
CHANGELOG: highlight "--log-outputs" flag
0e4c94c 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
Documentation/upgrades: highlight "--log-outputs"
22609a6 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
etcd.conf.yml.sample: rename to "log-outputs"
1539fdd 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
embed,etcdmain: rename to "--log-outputs" flag
af5bc43 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
functional: rename to "log-outputs"
a6c30ea 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
*: rename to "--log-outputs" in docs
58a6034 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
*: change typo in "auto-compaction-retention" field
b748abc 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
embed,integration,snapshot: use "LogOutputs"
88c70d0 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
gyuho added 2 commits April 25, 2018 13:52
@gyuho
pkg/logutil: do not build journald logger on windows
9cd9ae2 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho
embed: do not import journald dependencies on windows
154e2ac 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@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
# for free to join this conversation on GitHub. Already have an account? # 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