-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add Dockerfile for logrotate #76
base: develop
Are you sure you want to change the base?
Conversation
} | ||
|
||
# docker volume logs | ||
/var/lib/docker/volumes/*log*/_data/* { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to be more specific about this. User might have other containers running which have log in their name. We might end up rotating their files too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, why the location of directories is same as of the host 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/var/lib/docker/volumes/
, this directory will not work in case of macOS.
Also, this will skip the logs of EasyEngine itself.
For macOS a pattern inside ~/easyengine
for services
and sites
will have to be written.
} | ||
|
||
# docker volume logs | ||
/var/lib/docker/volumes/*log*/_data/* { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/var/lib/docker/volumes/
, this directory will not work in case of macOS.
Also, this will skip the logs of EasyEngine itself.
For macOS a pattern inside ~/easyengine
for services
and sites
will have to be written.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mrrobot47 We need to make this file logrotate/easyengine.conf
accessible on the host
issue: EasyEngine/feature-requests#73