-
-
Notifications
You must be signed in to change notification settings - Fork 138
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
make it compatible with Sonarr/Radarr V4 #252
Comments
can you share your logs so can see what is up? |
sure so here what I got for sonarr 2023-05-14 10:04:29 : DEBUG : init : Running job Job(interval=300, unit=seconds, do=thread, args=(<bound method SonarrAPI.get_queue of >,), kwargs={}) |
Greetings! As well for Radarr Saw from #251 looks like changes may have been merged to develop 3 days ago. Double checked and my container is update to date w/latest develops release but still seeing the errors above. Thanks for any assistance can provide/gladly provide any additional logs etc. -Cheers |
hey, thanks for the update in the develop branch -> boerderij/varken:develop but same as Healzangels, for Radarr: 2023-07-06 22:26:14 : DEBUG : connectionpool : http://********1.253:7878 "GET /api/v3/movie HTTP/1.1" 200 None but seem to be fixed for sonarr on my side thanks a lot :) |
Same issue here, I hope it will be fixed |
Here are my logs from the start of 01/01/24 if required: Both Sonarr and Radarr is affected. Varken-2024-01-01T23-03-11.log Sonarr and Radarr have now made it to v4 and v5 respectively on master branches. |
Hella late, but you guys can fix this yourself. I discovered the Line 207 in ec79d22
and add the following 2 lines (in alphabetical order to keep things easy to follow):
I'll update this if i find more, I had to use the radarr api here: https://radarr.video/docs/api/#/Movie/get_api_v3_movie in order to figure out the output from radarr. I'm rolling my own image for now, but I'll fork the develop branch and add the fix in there once I see if there are any more (I first did the root folder one, then got a new error for statistics, there could be more) fWIW, I'm using radarr v5, so not sure if my changes will work in older versions. Here is the start of my fork: evanrich/Varken@develop...evanrich-update-to-new-radarr-sonnar There are a bunch of things missing, I got passed radarr and now have sonarr errors. I'll do a PR -> develop branch once I get mine all sorted out I've opened a PR that fixes these errors #260 |
Thank you for fixing this. I built a docker image based on the #260. I confirmed that it works with the latest version(as of this writing) of Sonarr and Radarr, as well as support for Overseerr. I built and pushed my first docker image with these changes, so if anyone wants to pull it to use, it is: gokou360/varken:dev-v4api Thanks again @evanrich! |
No problem @gokou340 ! I also am now rolling my own image internally (hosting on a harbor instance in my cluster) since it looks like this will probably never get updated here. |
well as the title said, we are now with Sonarr/Radarr V4 so the queuelist doesnt work anymore
if anyone can update the code to work with the newer version
Thanks a lot
The text was updated successfully, but these errors were encountered: