-
Notifications
You must be signed in to change notification settings - Fork 36
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
Make startapp log file location configurable by an env var #129
Conversation
So that it is not included in the pushed backups
Hey, |
No worries. If there's utility in that behaviour then I can change it to be configurable via an env var then. |
We ask people to provide this logfile to see why there may be an issue with the app not starting/installing, etc and therefore we need a defined path to the logfile and that path should not change. |
I have no issues with myself (and presumably most others who explicitly set an environment variable) opening a shell into the Docker container and grabbing the log files out from the backend if necessary. As for excluding it in the Backblaze config, as far as I can tell this is not possible under the current setup. It is only possible to exclude folders and file extensions. Since this is put straight into the root of C:\ it is not possible to exclude it by folder and I do want |
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.
That is a good solution, thank you!
Please change the readme wording as in the comment.
Not sure why the CI is filing on this. Doesn't seem like anything related to the change |
yes, that's ecxpected. |
So that it is not included in the pushed backups