search

plexguide / PlexGuide.com

PlexGuide is a comprehensive Docker container management solution designed for Ubuntu & Debian OS. While it specializes in deploying media server applications like Plex, the platform incorporates Traefik as a reverse proxy and offers CloudFlare tunnel integration to access to your containerized services.
https://plexguide.com
1.72k stars 257 forks source link

Sab has been downloading to appdata #741

Closed ghost closed 5 years ago

ghost commented 5 years ago

I reinstalled sab with the latest 7.6edge and sab downloads to /opt/appdata!

Using linuxserver image

Admin9705 commented 5 years ago

hmm really? i will redeploy and check. what's storing there? everything?

ghost commented 5 years ago

Everything, it's odd. I deleted the folder from appdata and it starts dumping everything in there.

ghost commented 5 years ago

It's because downloads are under /config now??

So its basically a race condition if this works or not. Why was the download paths changed?

You also mount to /mnt/sab but /mnt/sabnzbd is created and empty.

The dl paths for the all downloaders and pvrs need redone.

You shouldn't use app default locations as they may differ between releases or docker images.

TechButton commented 5 years ago

I can also confirm with the new version I am also seeing issues on SAB and Radarr talking to each other. Radarr can see the movies that downloaded but they will not auto move anymore. I am working through the problem and see if I can find a solution currently.

TechButton commented 5 years ago

I did see there is now a temp directory that it is trying to download to first and then move to the other /config/downloads. I removed that temp folder right away, and didn't verify that was an issue. I will back up my sabnzbd and redeploy it again and try it out.

ghost commented 5 years ago

The downloads must be moved out of /config The mapping will cause issues since your mapping a parent and a child folder Having it like this causes weirdness with docker and it's.not.predictable. Sadly I just losty queues of 1tb of completed files. I'll have to rename and move normally.

The linuxserver dockerfile exposes volumes that differ

ghost commented 5 years ago

Also lazylibrarian was never updated with the new paths.

I would do a find/replace searching the entire project next time to be safer.

Admin9705 commented 5 years ago

sorry guys, will check into this, working something else. if you want to push things, please do :D working on a traefik google oauth solution

ghost commented 5 years ago

I should be able to help fix this tomorrow but I think we should fix all the paths once and for all. We should avoid mounting a parent and a subfolder in the mounted parent as things get goofy.

Also enable that oauth by default for blitzui

Admin9705 commented 5 years ago

@robertbaker sounds good, just push to 7.6 if u can. 7.7 is experimental with traefik right now. once it's 15 jan, i'll remove Edge and default the working project as in it would 7.6 now if it was the case. Thanks for your help and mentorship along the way. studying traefik more now, i see what you really did. i'm studying the backend piece along with the expose port. goal is to get the plex subdomain working. the authentication piece is working well, just held up by one piece.

ghost commented 5 years ago

This is what I'm thinking, this will be completely breaking but a script should be able to mv the old structure and adjust the config files.-

containers & app configs:

/incomplete/nzbget aka /incomplete/{{pgrole}} /downloads/nzbget /incomplete/sabnzbd /downloads/sabnzbd /incomplete/qbittorrent /downloads/qbittorrent [...]

instead of /mnt/sab,/mnt/nzbget....

/mnt/incomplete/nzbget aka {{path.stdout}}/incomplete/nzbget /mnt/downloads/nzbget /mnt/incomplete/sabnzbd /mnt/downloads/sabnzbd /mnt/incomplete/qbittorrent /mnt/downloads/qbittorrent [...]

This accomplishes a few things.

Then in sonarr/radarr/lazylibrarian/lidarr/etc you can just mount this once: {{path.stdout}}/downloads: /downloads

By having /mnt/downloads and /mnt/incomplete you can refactor this into vars - {{path.incomplete}} and {{path.downloads}} {{path.stdout}}

I'm going to start on a PR to do this. let me know what you think.

Admin9705 commented 5 years ago

Sounds like a plan

On Sat, Dec 29, 2018, 17:05 Robert Baker <notifications@github.com wrote:

This is what I'm thinking, this will be completely breaking but a script should be able to mv the old structure and adjust the config files.-

containers & app configs:

/downloads/incomplete/nzbget aka /downloads/incomplete/{{pgrole}} /downloads/nzbget /incomplete/sabnzbd /downloads/sabnzbd /incomplete/qbittorrent /downloads/qbittorrent [...]

instead of /mnt/sab,/mnt/nzbget....

/mnt/incomplete/nzbget /mnt/downloads/nzbget /mnt/incomplete/sabnzbd /mnt/downloads/sabnzbd /mnt/incomplete/qbittorrent /mnt/downloads/qbittorrent [...]

This accomplishes a few things.

  • Easy extendable pattern to support more downloaders and stuff in the future.
  • Allows a consolidated location that can be used for search/import/browsing, for example I could run du -h /mnt/downloads to get a print out of sizes for all of my downloaders.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Admin9705/PlexGuide.com-The-Awesome-Plex-Server/issues/741#issuecomment-450527024, or mute the thread https://github.com/notifications/unsubscribe-auth/AXlN3sQDuFT3gNrfAMRR_OzzwYCC7Jf8ks5u9_VOgaJpZM4ZjCdr .

ghost commented 5 years ago

@TechButton I have a PR to fix the paths once and for all out. I also took the time and created migration scripts and flushed out more initial config.

To fix your issue, you'll need to reinstall sabnzbd/nzbget,sonarr,radarr,lazylibrarian,deluge,rutorrent,qbittorrent. Basically the PVRs and downloaders. Plex and Ombi and such are fine.

You can either choose to uninstall the apps (warning deletes appconfigs) and install from scratch or you can install over the top and just make the edits needed through config files or through the web.

Hopefully my migration scripts work to move your downloads over to the new structure.

/mnt/downloads and /mnt/incomplete

/mnt/downloads/nzbget /mnt/incomplete/nzbget /mnt/incomplete/sabnzbd /mnt/downloads/sabnzbd [...] etc

This was done for various reasons that I can explain at a later time.

ghost commented 5 years ago

Just tested and sab and deluge are both working again for me.

The issue is with the downloaders and the PVRS the paths have to match on both sides of the mapping and they all must match each other or it doesn't work. When sab was changed to /config/Downloads, it no longer matched the left side and broke.

sonarr and the rest of the downloaders should work again now too.

Admin9705 commented 5 years ago

so i have this all pushed to 7.7 @robertbaker in edge. just going through testing. if you wanna double check 7.7 edge, it would help

Admin9705 commented 5 years ago

resolved with updates