-
Notifications
You must be signed in to change notification settings - Fork 189
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.
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
Enable OCIS to log to a file #1619
Comments
I don't think we should implement logging to a file. logging to stdout/stderr is already handled out of the box by docker and systemd. When starting it as a systemd unit stdout logging is already taken care of, see http://0pointer.de/blog/projects/journalctl.html
The official docker containers for nginx and apache even redirect their logfiles to stdout to let docker capture them, see https://docs.docker.com/config/containers/logging/ Logfiles seem to be a step back, don't they? |
Let me explain you the workflow we have for logs: Our daemons create log files that are stored in the local filesystem for a period of time (3 months). By having OCIS not having the possibility to write to a file when technically is possible, means that we need to configure another dependency (rsyslog or systemd) to accomplish that. Less external dependencies the better, etcd is also another one part of the picture. I thought to the goal was to run OCIS of the box, but this is currently not possible as many core functionalities are delegated to external services. |
Related discussion on Talk https://talk.owncloud.com/channel/infinitescale?msg=EXpyWL3vhPvi8uP8P |
Alex Unger commented: pr exists here: #1816 |
Alex Unger commented: OCIS File Logging Design Choices:
Use Cases
|
As per other software (nginx, apache, ...) and to make OCIS deployment more dev/admin friendly,
OCIS should have an option to log to a file.
Zerolog allows for that as we are using in in Reva so it comes out of the box.
Acceptance criteria
unit test for config option (don't test the logger :-P)The text was updated successfully, but these errors were encountered: