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

Add overseerr support #210

Closed
wants to merge 70 commits into from
Closed

Add overseerr support #210

wants to merge 70 commits into from

Conversation

redredbeard
Copy link

Added support for Overseer

It's setup a bit differently than Ombi due to the differences in their API's

Config Settings:
get_request_total_counts = true
Endpoint - /api/v1/request
Gets the totals of TV/Movie Requests. They paginate automatically so I set an arbitrary high value for this endpoint instead of issuing multiple calls to get the pages (9999999999)

request_total_run_seconds = 300
Runs every N seconds

get_request_status_counts = true
Endpoint - /api/v1/request/count
Gets the status counts (Pending/Approved/Processing/Available) of all requests

request_status_run_seconds = 300
Runs every N seconds

get_latest_requests = true
num_latest_requests_to_fetch = 10
Endpoint - /api/v1/request
Movie Endpoint - /api/v1/movie/{movieID}
TV Endpoint - /api/v1/tv/{tvID}

No actual movie/tv data is returned with the request endpoint, instead it returns a TMDB id, which then you have to make a separate API call to get the title/name for. To prevent a large amount of API calls every n mins, I added an option to limit the amount of recent requests to pull. This won't pull your full request history, only the latest n entries.

num_latest_requests_seconds = 300
Runs every N seconds

API documentation: https://api-docs.overseerr.dev/

dirtycajunrice and others added 30 commits February 3, 2019 23:53
v1.6.4 Merge
v1.6.7 Merge
v1.6.8 Merge
v1.7.0 Merge
v1.7.2 Merge
v1.7.3 Merge
v1.7.5
Merge pull request #191 from Boerderij/develop
This contains all the data needed for connecting and writing to an InfluxDB2 server
This stores the data needed for InfluxDB2, and has a single `write_points` function on this that takes an array of points to add to the database
@cipi1965

This comment has been minimized.

@samwiseg0

This comment has been minimized.

Comment on lines +12 to +16
<<<<<<< HEAD
from varken.structures import SonarrServer, RadarrServer, OmbiServer, OverseerrServer, TautulliServer, InfluxServer, Influx2Server
=======
from varken.structures import SonarrServer, RadarrServer, OmbiServer, OverseerrServer, TautulliServer, InfluxServer
>>>>>>> 07cc5946f37b14ab268c795fbdc5092988c10e97

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merge conflict

Comment on lines +129 to +133
<<<<<<< HEAD
=======
print(result)

>>>>>>> 07cc5946f37b14ab268c795fbdc5092988c10e97

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merge conflict

Comment on lines 81 to 84
<<<<<<< HEAD
=======

>>>>>>> 07cc5946f37b14ab268c795fbdc5092988c10e97

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merge conflict

@snorkrat
Copy link

Would love to see this merged. I've been running t0phux's version for months now with no issues at all.

@darkcloud784

This comment has been minimized.

@samwiseg0 samwiseg0 changed the base branch from master to develop December 24, 2021 06:29
@rjdodd00

This comment has been minimized.

@samwiseg0
Copy link
Member

samwiseg0 commented Jan 15, 2022

In the future please don't PR two major things in one PR. It makes is difficult to go through it.

I would have preferred that this PR be Overseerr support only. Not influx2 and overseerr in one PR

I am going to see if I can clean this up a bit and get it merged but its a lot of changes.

samwiseg0 added a commit that referenced this pull request Jan 15, 2022
@samwiseg0
Copy link
Member

I was able to prune your changes from the influxdb2 changes. Overseerr support is now live in develop.

Please test and let us know if there are any issues!

"status": get_movie_req['mediaInfo']['status'],
"title": get_movie_req['title'],
"requested_user": get_movie_req['mediaInfo']['requests'][0]['requestedBy']['plexUsername'],
"requested_date": get_movie_req['mediaInfo']['requests'][0]['requestedBy']['createdAt']

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"requested_date": get_movie_req['mediaInfo']['requests'][0]['requestedBy']['createdAt']
"requested_date": get_movie_req['mediaInfo']['requests'][0]['media']['createdAt']

Actually it's grabbing "createdAt" of user not media

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes have been included in the latest commit on develop.

"status": get_tv_req['mediaInfo']['status'],
"title": get_tv_req['name'],
"requested_user": get_tv_req['mediaInfo']['requests'][0]['requestedBy']['plexUsername'],
"requested_date": get_tv_req['mediaInfo']['requests'][0]['requestedBy']['createdAt']

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"requested_date": get_tv_req['mediaInfo']['requests'][0]['requestedBy']['createdAt']
"requested_date": get_tv_req['mediaInfo']['requests'][0]['media']['createdAt']

Actually it's grabbing "createdAt" of user not media

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes have been included in the latest commit on develop.

@Animizio

This comment was marked as off-topic.

@samwiseg0

This comment was marked as off-topic.

@samwiseg0
Copy link
Member

Going to close this as it's been implemented in develop.

@samwiseg0 samwiseg0 closed this Jul 12, 2022
MDHMatt pushed a commit to MDHMatt/Biggetje that referenced this pull request May 12, 2023
d-mcknight added a commit to d-mcknight/Varken that referenced this pull request Jun 22, 2023
* Boerderij#203

* Update docker compose to specify influxdb:1.8.4

* Update requirements to use urllib3==1.26.5

* updated to support Radarr and Sonarr V3 Api

* bump requirements for requests

* Fix Sonarr & Radarr V3 API /queue endpoint (Boerderij#220)

* Fix lint issues

* More lint fixes

* Update Sonarr structures

* Add Overseerr Support (Boerderij#210)

* Remove duplicate structures

* update changelog to reflect v1.7.7 changes

* Add IP data to tautulli Boerderij#202

* add missing ip address in tautulli

* Fixed: Streamlined API calls to Radarr and Sonarr (Boerderij#221)

* Fixed: Sonarr Data pull issues (Boerderij#222)

* Fix Sonarrr calendar

* Update lidarr structure (Boerderij#225)

Added missing arguments to Lidarr structure

Fixes Boerderij#223

* Clean up request totals. Upstream change sct/overseerr#2426

* Cleanup blank space

* Fix requested_date syntax.

* Fix requested_date for Overseerr tv and movie

* Fix overseerr config refernces

* Fix overseerr structures

* Update intparser to accommodate changes to config structure

* Cleanup overseerr data collection

* Fix SERVICES_ENABLED in varken.py to acomidate overseerr

* Fixed: Sonarr/Lidarr Queues (Boerderij#227)

* Change sonarr queue structures to str

* Fixed: Multipage queue fetching

* Update historical tautulli import (Boerderij#226)

* Fixed: Sonarr perams ordering

* Fixed: Proper warnings for missing data in sonarr and radarr

* Added: Overseerr ENVs to docker compose.

* Added: Logging to empty/no data returns

* Update Sonarr & Lidarr Structs to match latest API changes (Boerderij#231)

* Add support for estimatedCompletionTime in LidarrQueue

* Add support for tvdbId in SonarrEpisode struct

* Fix typo in docker yml

* Rename example url for overseerr in docker yml

* Update radarr structures to inclue originalLanguage

* Update radarr structures to include addOptions

* Update radarr structures to include popularity

* fix(ombi): Update structures.py (Boerderij#238)

* feat(docker): remove envs from example

* fix(logging): remove depreciation warning. Var for debug mode (Boerderij#240)

* Support InfluxDB 2.x as addition to 1.8

* Document that influxdb 2.x is supported

* Include influxdb username/password for v2 server support

* Support an optional v prefix for influxdb version strings

---------

Co-authored-by: mal5305 <malcolm.e.rogers@gmail.com>
Co-authored-by: samwiseg0 <2241731+samwiseg0@users.noreply.github.com>
Co-authored-by: Robin <19610103+RobinDadswell@users.noreply.github.com>
Co-authored-by: tigattack <10629864+tigattack@users.noreply.github.com>
Co-authored-by: Stewart Thomson <stewartthomson3@gmail.com>
Co-authored-by: Cameron Stephen <mail@cajs.co.uk>
Co-authored-by: MDHMatt <10845262+MDHMatt@users.noreply.github.com>
Co-authored-by: Nathan Adams <dinnerbone@dinnerbone.com>
d-mcknight added a commit to d-mcknight/Varken that referenced this pull request Jun 23, 2023
* Boerderij#203

* Update docker compose to specify influxdb:1.8.4

* Update requirements to use urllib3==1.26.5

* updated to support Radarr and Sonarr V3 Api

* bump requirements for requests

* Fix Sonarr & Radarr V3 API /queue endpoint (Boerderij#220)

* Fix lint issues

* More lint fixes

* Update Sonarr structures

* Add Overseerr Support (Boerderij#210)

* Remove duplicate structures

* update changelog to reflect v1.7.7 changes

* Add IP data to tautulli Boerderij#202

* add missing ip address in tautulli

* Fixed: Streamlined API calls to Radarr and Sonarr (Boerderij#221)

* Fixed: Sonarr Data pull issues (Boerderij#222)

* Fix Sonarrr calendar

* Update lidarr structure (Boerderij#225)

Added missing arguments to Lidarr structure

Fixes Boerderij#223

* Clean up request totals. Upstream change sct/overseerr#2426

* Cleanup blank space

* Fix requested_date syntax.

* Fix requested_date for Overseerr tv and movie

* Fix overseerr config refernces

* Fix overseerr structures

* Update intparser to accommodate changes to config structure

* Cleanup overseerr data collection

* Fix SERVICES_ENABLED in varken.py to acomidate overseerr

* Fixed: Sonarr/Lidarr Queues (Boerderij#227)

* Change sonarr queue structures to str

* Fixed: Multipage queue fetching

* Update historical tautulli import (Boerderij#226)

* Fixed: Sonarr perams ordering

* Fixed: Proper warnings for missing data in sonarr and radarr

* Added: Overseerr ENVs to docker compose.

* Added: Logging to empty/no data returns

* Update Sonarr & Lidarr Structs to match latest API changes (Boerderij#231)

* Add support for estimatedCompletionTime in LidarrQueue

* Add support for tvdbId in SonarrEpisode struct

* Fix typo in docker yml

* Rename example url for overseerr in docker yml

* Update radarr structures to inclue originalLanguage

* Update radarr structures to include addOptions

* Update radarr structures to include popularity

* fix(ombi): Update structures.py (Boerderij#238)

* feat(docker): remove envs from example

* fix(logging): remove depreciation warning. Var for debug mode (Boerderij#240)

* fix(build): bump schedule version to 1.1

* fix(build): bump docker python version

* fix(dep): update requests and urllib3

* fix(sonarr): ensure invalid sonarr queue items are just skipped over - fixes Boerderij#239 (Boerderij#243)

* add branch to build inputs

* update pipeline badge

* Update automation

* Add influxdb 2 client

* Add structure for influxdb 2 params

This contains all the data needed for connecting and writing to an InfluxDB2 server

* Parse influxdb 2 config data

* Add influxdb2 manager class

This stores the data needed for InfluxDB2, and has a single `write_points` function on this that takes an array of points to add to the database

* Use the correct db manager for varken

* Add influxdb2 to the example varken config file

* Create influx bucket if it doesn't exist

* Update InfluxDB type on README

* Clean up linting errors

* Wrap create bucket in try/catch

* Use bucket given in ini file

* Log exception to troubleshoot errors

* Allow configured influx2 address as URL (no port)

* Bypass validity check to troubleshoot

---------

Co-authored-by: mal5305 <malcolm.e.rogers@gmail.com>
Co-authored-by: samwiseg0 <2241731+samwiseg0@users.noreply.github.com>
Co-authored-by: Robin <19610103+RobinDadswell@users.noreply.github.com>
Co-authored-by: tigattack <10629864+tigattack@users.noreply.github.com>
Co-authored-by: Stewart Thomson <stewartthomson3@gmail.com>
Co-authored-by: Cameron Stephen <mail@cajs.co.uk>
Co-authored-by: MDHMatt <10845262+MDHMatt@users.noreply.github.com>
Co-authored-by: Nathan Adams <dinnerbone@dinnerbone.com>
Co-authored-by: Nicholas St. Germain <nick@cajun.pro>
Co-authored-by: Gabe Revells <gcrevell@mtu.edu>
ajanis added a commit to ajanis/Varken that referenced this pull request Jun 26, 2023
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

Successfully merging this pull request may close these issues.

None yet