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.

Sign up for GitHub

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

Jump to bottom

[BUG] TV requests not showing up in official Grafana dashboard with Ombi v4 #203

Open
mal5305 opened this issue May 19, 2021 · 0 comments
Open

[BUG] TV requests not showing up in official Grafana dashboard with Ombi v4 #203

mal5305 opened this issue May 19, 2021 · 0 comments
Labels
approved Enhancement or request that was approved to be worked on bug Something isn't working in-next-release In the next release

Comments

@mal5305
Copy link

mal5305 commented May 19, 2021

TV requests not showing up in official Grafana dashboard "Ombi Requests Pending" table. "Requests Approved" singlestat does reflect correct number of total requests.

Steps to reproduce the behavior:

  1. add TV request to Ombi (v4)
  2. use official Varken grafana dashboard (1.6.x)

Expected behavior
TV request should show up in Grafana panel/table

Environment:

  • boerderij/varken:latest (docker)
  • Ombi 4.0.1342 (docker)
  • influxdb v1.7.2

looks like an Ombi API change. API is returning a field called "externalProviderId" (int) which is not present in OmbiTVRequest class structure
@mal5305 mal5305 added the awaiting-triage Request awaiting triage label May 19, 2021
mal5305 added a commit to mal5305/Varken that referenced this issue May 21, 2021
@mal5305
Boerderij#203
2b1f6e0
samwiseg0 added a commit that referenced this issue May 21, 2021
@samwiseg0
Merge pull request #205 from mal5305/#203
01b7d2c
@samwiseg0 samwiseg0 added approved Enhancement or request that was approved to be worked on bug Something isn't working in-next-release In the next release and removed awaiting-triage Request awaiting triage labels May 23, 2021
MDHMatt pushed a commit to MDHMatt/Biggetje that referenced this issue May 12, 2023
@mal5305 @MDHMatt
Boerderij#203
b6e53db
d-mcknight added a commit to d-mcknight/Varken that referenced this issue Jun 22, 2023
@d-mcknight @mal5305
@samwiseg0 @RobinDadswell @tigattack @StewartThomson @Cajs @MDHMatt @Dinnerbone
Influxdb 2 (#1)
555f7d1 
* 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 <[email protected]>
Co-authored-by: samwiseg0 <[email protected]>
Co-authored-by: Robin <[email protected]>
Co-authored-by: tigattack <[email protected]>
Co-authored-by: Stewart Thomson <[email protected]>
Co-authored-by: Cameron Stephen <[email protected]>
Co-authored-by: MDHMatt <[email protected]>
Co-authored-by: Nathan Adams <[email protected]>
d-mcknight added a commit to d-mcknight/Varken that referenced this issue Jun 23, 2023
@d-mcknight @mal5305
@samwiseg0 @RobinDadswell @tigattack @StewartThomson @Cajs @MDHMatt @Dinnerbone @dirtycajunrice @gcrevell
Influxdb2 (#3)
7e03144 
* 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 <[email protected]>
Co-authored-by: samwiseg0 <[email protected]>
Co-authored-by: Robin <[email protected]>
Co-authored-by: tigattack <[email protected]>
Co-authored-by: Stewart Thomson <[email protected]>
Co-authored-by: Cameron Stephen <[email protected]>
Co-authored-by: MDHMatt <[email protected]>
Co-authored-by: Nathan Adams <[email protected]>
Co-authored-by: Nicholas St. Germain <[email protected]>
Co-authored-by: Gabe Revells <[email protected]>
ajanis pushed a commit to ajanis/Varken that referenced this issue Jun 26, 2023
@mal5305 @ajanis
Boerderij#203
1ed80f7
MDHMatt added a commit to MDHMatt/Biggetje that referenced this issue Aug 7, 2024
@MDHMatt @mal5305
@samwiseg0 @RobinDadswell @tigattack @StewartThomson @Cajs @Dinnerbone @wtf0722 @douginoz
Develop to master v2 (#7)
357895e 
* 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)

* Update structures.py

watchedByRequestedUser error

* changed repo, changed maintainer, referenced original

* rename action

* Add runtime keyword to SonarrEpisode structure (Boerderij#251)

* Bump requests to 2.31.0

* Update structures.py for latest Ombi typeerror on playedByUsersCount and watchedByRequestedUser (Boerderij#256)

* Update action.yml

---------

Co-authored-by: mal5305 <[email protected]>
Co-authored-by: samwiseg0 <[email protected]>
Co-authored-by: Robin <[email protected]>
Co-authored-by: tigattack <[email protected]>
Co-authored-by: Stewart Thomson <[email protected]>
Co-authored-by: Cameron Stephen <[email protected]>
Co-authored-by: Nathan Adams <[email protected]>
Co-authored-by: wtf0722 <[email protected]>
Co-authored-by: Dir <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
approved Enhancement or request that was approved to be worked on bug Something isn't working in-next-release In the next release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@samwiseg0 @mal5305