Send zookeeper logs to standard out

[mannytoledo]

Redirecting zookeeper output to stdout to make it easier to monitor with standard docker tools.

[brandongalbraith]

@mannytoledo Could this be optional and have the linking be done as a conditional if an environment variable is passed into the container?

[mannytoledo]

@brandongalbraith Is there a use case where a user wouldn't want zookeeper logs accessible from the running container?

I originally though of making it optional but figured the app this container wraps should be accessible via docker logs.

[brandongalbraith]

@mannytoledo Not sure about a use case where you wouldn't want logs; I'm thinking about people already using this in production who are expecting a certain behavior (logs not send to stdout).

[mannytoledo]

@brandongalbraith Fair enough, expect an update on this soon.

[brandongalbraith]

Thanks @mannytoledo!

[mannytoledo]

I went ahead and made that optional @brandongalbraith @mbabineau

Let me know what you guys think.

[brandongalbraith]

@mannytoledo 👍

[panda87]

Thanks @mannytoledo for this PR.
Someone has an idea how to set logrotate to zk logs?
Im using this repo and my zookeeper.out log is now a huge file and I dont know how to set logrotate for that.

[brandongalbraith]

@panda87

See http://www.jamescoyle.net/cheat-sheets/676-logrotate-cheat-sheet on how to logrotate zookeeper.out. A suggestion would be:

/opt/zookeeper/zookeeper.out {
  daily
  rotate 7
  compress
  copytruncate
  missingok
}

placed in a file such as /etc/logrotate.d/zookeeper. This would rotate logs daily, compress rotated logs, and keep seven days of log history. It also prevents errors if the log doesn't exist.

[panda87]

Thanks @brandongalbraith. I will add that to my docker container

[brandongalbraith]

@panda87 You'd set that up on your host running the docker container, not
within the docker container.

On Monday, May 2, 2016, panda87 [email protected] wrote:

Thanks @brandongalbraith https://github.com/brandongalbraith. I will
add that to my docker container

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#14 (comment)

[panda87]

This is the best practice? because the json file log is located in the /var/lib/docker dir