Change syslog format and facility#12391
Conversation
|
Do we need a test involving |
|
@ibuildthecloud I'm -1 on names by default as @phemmer , and I have same concerns as @thaJeztah . I think that there is more usecases about IDs, then about Names. But I'm +1 on |
|
I'll like @phemmer format, but still want container names. I think we can agree that there are use cases for both. We will make it configurable, I'll personally put in the PR for that, but 1.6 I still think container name is more friendly. |
a21f293 to
b760379
Compare
|
I update the PR to be "docker/NAME" |
|
Okay, my thoughts about id-name war:
|
This patch changes two things 1. Set facility to LOG_DAEMON 2. Remove ": " from tag so that the tag + pid become a single column in the log Signed-off-by: Darren Shepherd <[email protected]>
b760379 to
05641cc
Compare
|
Okay, I'm taking my toys and going home. I updated the PR to do container ID. So the PR just changes the facility and then the final format will be "docker/ID" looking like: |
|
@thaJeztah now that the format is ID, i don't think a rename test is really needed. |
|
sorry :( |
|
@ibuildthecloud not as part of this PR, no. But I think it will be needed for the 1.7 (1.6.1?) update that supports I realise the outcome here is not what you hoped for (I tried, within reason), but given the short time before the release, I think it's reasonable. "Go go gadget |
|
LGTM |
1 similar comment
|
LGTM |
Change syslog format and facility
|
ok cherry-picked into release bump |
This patch changes three things
the log
and hard to correlate with events in the past
This is an implementation of the my comments on #12377
In the end running
docker run --name test --rm --log-driver syslog ubuntu echo hiwill yield the following log in UbuntuFix #12377