Sperryfreak01 / RadarrSync

120 stars 46 forks source link

4 radarrs? #1

Closed ghost closed 6 years ago

ghost commented 6 years ago

Can you use more than 2?

Sperryfreak01 commented 6 years ago

Today it's limited to two servers, and will only sync profile '5' (ultrahd).

Wouldn't be hard to expand it to more servers though. Can you give me your use case so I can think about it.

Sperryfreak01 commented 6 years ago

Actually in a crude way it does support one-to-many. Just create multiple copies of the script in different folders, only one config.txt file per folder, and run those but it is hard coded to ultrahd right now.

hjone72 commented 6 years ago

@danjames92, could you please test https://github.com/hjone72/RadarrSync/tree/MultiServer before I make a pull request?

EnorMOZ commented 6 years ago

@hjone72 I just tried and noticed 2 things... The path mappings don't see to be working. It seems like it is pulling everything. Also it did not like chinese characters... ie: When I set the path mapping of: /mnt/unionfs/Media/Kids Movies to /mnt/unionfs/Media/Kids Movies4k It was pulling stuff from /mnt/unionfs/Media/Movies to /mnt/unionfs/Media/Movies/`

UnicodeEncodeError: 'ascii' codec can't encode character u'\xb3' in position 31: ordinal not in range(128)

It also did not add the correct profile id. It added it as 4 vs 5. profileId = 5 profileIdMatch = 4

From master server where profile are 1080 to second radarr for 4k.

hjone72 commented 6 years ago

@EnorMOZ would you be able to post your Output.txt please? Feel free to remove any sensitive data.

hjone72 commented 6 years ago

@EnorMOZ I've done a bit of testing using the following characters 東京ゴッドファーザーズ. I can't seem to fault the program anymore.

The code I added was Python2 not 3. Python3 handles these characters much better. I've updated the code to work with Python3. Grab the latest commit and test again please?

Edit: Found an error that didn't appear in the dev environment, when the logger is called it is throwing an error. The actual sync is working though. Working that out now.

EnorMOZ commented 6 years ago

@hjone72

Example output: Clearly not a "kids movie" :) 2018-03-31 05:27:02,130 [MainThread ] [INFO ] adding 300: Rise of an Empire to Radarr 4k server

It is almost like it is ignoring the config basepath and using the first one in radarr which is set to /mnt/unionfs/Media/Movies.

Trying to sync the movies from master which are set to profile 4 from the basepath of /mnt/unionfs/Media/Kids\ Movies to Radarr_4k setting it to profile 5 to the basepath of /mnt/unionfs/Media/Kids\ Movies4k

Example config file. `[RadarrMaster] url = http://127.0.0.1:7878 key = XXXXX basepath = /mnt/unionfs/Media/Kids\ Movies

[SyncServers] ; Ensure the servers start with 'Radarr_' [Radarr_4k] url = http://127.0.0.1:7879 key = XXXXXXXX newpath = /mnt/unionfs/Media/Kids\ Movies4k ;This is the profile ID the movie will be added to. profileId = 5 ; This is the profile ID the movie must have on the Master server. profileIdMatch = 4`

I am in the plexdrive slack channel if you ever want to troubleshoot this in realtime.

EnorMOZ commented 6 years ago

@hjone72, Take a look at my fork. It is my attempt to try to sling python. The path part seems to be working but images are not working properly.

lcasey001 commented 6 years ago

https://github.com/hjone72/RadarrSync/tree/MultiServer works great now.

Sperryfreak01 commented 6 years ago

fixed by #7, multiple servers are now supported. No restriction on the number or naming of server, as long as the primary server is named 'Radarr' or 'default'