-
Notifications
You must be signed in to change notification settings - Fork 132
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 role: transmissionx #273
Conversation
Out of curiosity, why not just do the |
Well... I don't really have a reason, I just went with it... It's a good idea though. Removed transmission leaving only transmissionx |
…hub.com\linuxserver\docker-transmission\issues\100
… moving download dir to /downloads/
It works fine for me, I've tried it on a somewhat new Cloudbox install and on a completely fresh one. There is a minor issue but I don't think it's solvable: When the container starts even if the settings.json file has the correct configuration the container will automatically create the /downloads/complete and /downloads/incomplete directories. I've tested mapping /downloads/ to /mnt/local/downloads/torrents/transmission{role}/ but this creates issues with sonarr because the path mapping does not match. In my opinion this is minor because it's just an empty folder. I'll note this in the wiki but I think this is mergeable now. |
This PR seems like a bit of a mess. Why is the opt folder mounted as
And you mount the download folders twice? |
Well I'm glad it is a bit of a mess and not a complete mess hahaha. opt is mounted like that because I copied from https://github.com/Cloudbox/Community/blob/develop/roles/transmissionvpn/tasks/main.yml and used the "if it ain't broke, don't fix it" rule. as for this double mount
It is because the container automatically changes the watched folder at each start and sets it to /watch so this is the only solution. See: linuxserver/docker-transmission#100 |
By double mount I meant
You define the download mount yet that is already included by the default setup. |
As far as copying the vpn version that is not a good way of doing things when you're not using the same image. |
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.
This would be my suggestion for the role. Take it for a spin?
Everything seems to be working fine, both the already in use updated roles and the new ones I've put for testing |
This adds transmission, my personal favorite torrent client. Capable of handling 20k+ torrents without much trouble even in gui / desktop client.
suggestions and testing welcome