-
-
Notifications
You must be signed in to change notification settings - Fork 105
feat(role): add rutorrent (role-refactor) #505
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
base: role-refactor
Are you sure you want to change the base?
Conversation
| rutorrent_role_docker_volumes_download: false | ||
| rutorrent_role_docker_volumes_default: | ||
| - "{{ rutorrent_role_paths_location }}:/config" | ||
| - "{{ lookup('role_var', '_topdir_path', default=rutorrent_role_paths_downloads_location) }}:/data:ro" |
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.
read only?
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.
Nothing uses /data since TOPDIR_PATH and DOWNLOAD_PATH are both set to /mnt paths, but without this mount the image creates an anonymous volume for it (same with /passwd). ro prevents the init script creating unused directories in there, which I noticed happens on occasion.
Description
https://hub.docker.com/r/k44sh/rutorrent
Role should provide a good foundation for all crazy-max based images in case this one goes unmaintained.
How Has This Been Tested?
I'm expecting this will clash in CI with the existing role, but yoloing it.