deevus / docker-plex-remote-transcoder

46 stars 10 forks source link

Update and build improvements #8

Closed oaxlin closed 7 years ago

oaxlin commented 7 years ago

I tried to make sure setup of a BRAND NEW master/slave was a lot easier to get working.

1) Updates to fix some oddness with Library linking. Works with Plex Version 1.7.5.4035 using debian docker slave with synology docker master

2) Reorders Dockerfile to enable re-builds to go faster by making use of the docker cache

3) Docker startup times are dramatically improved by moving the base prt installation into the Dockerfile

4) Now you don't need to setup the known_hosts file, it is automatic. Note: if known_host is giving you issues simply delete it and let it auto-rebuild

5) SSH performance between master/slave is improved by enabling the ControlPersist feature

The only real problem I didn't solve is verifying permissions are working properly between master->master_host->slave_host->slave. It's kinda tricky to get those 100% perfect unless you run some sort of LDAP or AD environment. I'm currently trying to think of ways to test permissions and at least warn of potential problems for new users. If I do figure something out, then I'll end up sending you another pull request.

deevus commented 7 years ago

Looks good. Thank you