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

Cannot change watch folder via settings.json #100

Closed
zilexa opened this issue Apr 12, 2020 · 7 comments
Closed

Cannot change watch folder via settings.json #100

zilexa opened this issue Apr 12, 2020 · 7 comments

Comments

@zilexa
Copy link

zilexa commented Apr 12, 2020


Expected Behavior

You should be able to change settings via the settings.json file, only modifying it after stopping the container.

Current Behavior

All modified settings are present EXCEPT watch-dir, it's value is back to default "/watch".

Steps to Reproduce

  1. Stop the container.
  2. Wait until it is stopped.
  3. Edit the config file via the folder that was mounted during container creation, ie: /$home/docker/transmission/config/settings.json
  4. Edit multiple values including watch-dir.
  5. Save.
  6. Start the container, wait till its started.
  7. Open webUI to check if its running.
  8. Open settings.json again to verify modifications persisted.

Environment

OS: Ubuntu 19.10
CPU architecture: x86_64
How docker service was installed:docker-compose.yml

Command used to create docker container)

  transmission:
    image: linuxserver/transmission
    container_name: transmission
    network_mode: service:vpn-proxy
    depends_on:
      - vpn-proxy
    restart: unless-stopped
    environment:
      - PUID=${PUID}
      - PGID=${PGID}
      - TZ=Europe/Amsterdam
      - TRANSMISSION_WEB_HOME=/transmission-web-control/
      - USER=admin
      - PASS=removed
    volumes:
      - ${USERDIR}/docker/transmission/config:/config
      - ${USERDIR}/Media:/media
@aptalca
Copy link
Member

aptalca commented Apr 12, 2020

You need to use the volume mapping for the watch folder. We hardcode it in the app start arguments: https://github.com/linuxserver/docker-transmission/blob/master/root/etc/services.d/transmission/run#L15

@zilexa
Copy link
Author

zilexa commented Apr 12, 2020

ah, but why?
I use /media:/media for all apps to prevent the issue of having multiple docker filesystems.
By using volume mapping for this item, I get an extra file system in docker.

@adripo
Copy link

adripo commented Apr 17, 2020

@zilexa where is located your watch folder?
Is it a problem declare an extra volume in your docker compose file?
It is not an extra file system, just an extra link to your existing file system.
Remember you also need to specify the download folder.
Something like that:

volumes:
      - ${USERDIR}/docker/transmission/config:/config
      - ${USERDIR}/Media/downloads:/downloads
      - ${USERDIR}/Media/watch:/watch

@zilexa
Copy link
Author

zilexa commented Apr 17, 2020

Actually I did that exactly and it still doesn't work. the settings.json file shows "".

@adripo
Copy link

adripo commented Apr 19, 2020

Have you checked the permissions on that file? Try removing it and let the container recreate it when it starts.
Please also check if the uid of the user you are using to read the settings file is the same as the one you pass to the container as env, or at least if it has the same permissions.

@zilexa
Copy link
Author

zilexa commented May 13, 2020

Not sure what I did wrong but it seems to work now.

@elisimpson
Copy link

What if we want to disable the watch dir entirely?

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

4 participants