Skip to content
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

logging #6

Open
jameseck opened this issue Dec 14, 2016 · 3 comments
Open

logging #6

jameseck opened this issue Dec 14, 2016 · 3 comments

Comments

@jameseck
Copy link

Hi,

Thanks for the docker image - it works beautifully. I'm running this on an openshift cluster and it's great to see someone make a docker image that doesn't require root.

One suggestion I would like to make is some consideration for logging. It would be great if this could be configured to log to stdout (docker logs).

I'm not sure of the best method of doing this, just thought I'd throw it out here. Maybe a wrapper script as pid1 that tails the log file as well as starting qbittorrent would be sufficient.

@wernight
Copy link
Owner

wernight commented Dec 16, 2016

I see basic logging is on stdout, but the rest is on /home/qbittorrent/.local/share/data/qBittorrent/logs/.

Quick browsing didn't show an official way to redirect to stdout. It might be possible to redirect using a symbolic link to /dev/stdout, but it might not work as it's making .bak files.

Unless someone finds a working solution, I'd suggest to mount to cat those log files when needed (not the nicest I agree).

@jameseck
Copy link
Author

Yeah I couldn't find a nice solution either. I might knock up a PR using supervisord if that's something you'd accept. It slightly breaks the one process per container model, but I don't personally stick to that when there isn't a nice way to achieve things.

@wernight
Copy link
Owner

I wouldn't accept a supervisord or initd on this image as it breaks its philosophy. It's worth trying doing a simple ln -s and see if it works, and possibly there is a config that avoids these backups. If not, it's always possible to send a PR to qBT to include a log to stdout CLI flag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants