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

QualityProfile does not exist on Radarr-4K #5025

Closed
ExtremeFiretop opened this issue Oct 16, 2023 · 9 comments
Closed

QualityProfile does not exist on Radarr-4K #5025

ExtremeFiretop opened this issue Oct 16, 2023 · 9 comments

Comments

@ExtremeFiretop
Copy link
Contributor

ExtremeFiretop commented Oct 16, 2023

Summary

When using 2 instances of Radarr (Radarr and Radarr-4K), if you delete/rename the 4K instances profiles, Ombi can no longer connect and request content.

Steps to re-create:

  1. Deleting all profiles except "Ultra-HD" on the second instance of Radarr (Radarr-4K)
  2. Then rename that one remaining profile on Radarr-4K
  3. Ombi can no longer use that the one single remaining (and renamed profile)

image

Ombi Version

4.43.5

What platform(s) does this occur on?

Windows

What database are you using?

SQLite (Default)

Relevant log output

2023-10-15 19:32:28.881 -04:00 [WRN] Tried auto sending movie but failed. Message: QualityProfile does not exist
2023-10-15 19:32:28.881 -04:00 [WRN] Tried auto sending movie but failed. Message: QualityProfile does not exist
2023-10-15 19:34:27.049 -04:00 [ERR] Something bad happened, ErrorMiddleware caught this
Octokit.RateLimitExceededException: API rate limit exceeded for 142.198.127.197. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)
   at Octokit.Connection.HandleErrors(IResponse response) in /home/runner/work/octokit.net/octokit.net/Octokit/Http/Connection.cs:line 704
   at Octokit.Connection.RunRequest(IRequest request, CancellationToken cancellationToken) in /home/runner/work/octokit.net/octokit.net/Octokit/Http/Connection.cs:line 685
   at Octokit.Connection.Run[T](IRequest request, CancellationToken cancellationToken) in /home/runner/work/octokit.net/octokit.net/Octokit/Http/Connection.cs:line 670
   at Octokit.ApiConnection.GetPage[TU](Uri uri, IDictionary`2 parameters, String accepts, ApiOptions options) in /home/runner/work/octokit.net/octokit.net/Octokit/Http/ApiConnection.cs:line 628
   at Octokit.ApiConnection.<>c__DisplayClass18_0`1.<<GetAll>b__0>d.MoveNext() in /home/runner/work/octokit.net/octokit.net/Octokit/Http/ApiConnection.cs:line 212
--- End of stack trace from previous location ---
   at Octokit.ApiPagination.GetAllPages[T](Func`1 getFirstPage, Uri uri)
   at Ombi.Schedule.Processor.ChangeLogProcessor.GetGitubRelease(Release release, Task`1 settingsTask) in /home/runner/work/Ombi/Ombi/src/Ombi.Schedule/Processor/ChangeLogProcessor.cs:line 71
   at Ombi.Schedule.Processor.ChangeLogProcessor.Process() in /home/runner/work/Ombi/Ombi/src/Ombi.Schedule/Processor/ChangeLogProcessor.cs:line 36
   at Microsoft.Extensions.Caching.Memory.CacheExtensions.GetOrCreateAsync[TItem](IMemoryCache cache, Object key, Func`2 factory)
   at Ombi.Helpers.CacheService.GetOrAddAsync[T](String cacheKey, Func`1 factory, DateTimeOffset absoluteExpiration) in /home/runner/work/Ombi/Ombi/src/Ombi.Helpers/CacheService.cs:line 29
   at Ombi.Controllers.V1.UpdateController.UpdateAvailable() in /home/runner/work/Ombi/Ombi/src/Ombi/Controllers/V1/UpdateController.cs:line 27
   at lambda_method3774(Closure , Object )
   at Microsoft.AspNetCore.Mvc.Infrastructure.ActionMethodExecutor.AwaitableObjectResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, Object controller, Object[] arguments)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.<InvokeActionMethodAsync>g__Awaited|12_0(ControllerActionInvoker invoker, ValueTask`1 actionResultValueTask)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.<InvokeNextActionFilterAsync>g__Awaited|10_0(ControllerActionInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Rethrow(ActionExecutedContextSealed context)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.<InvokeInnerFilterAsync>g__Awaited|13_0(ControllerActionInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeNextResourceFilter>g__Awaited|25_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Rethrow(ResourceExecutedContextSealed context)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeFilterPipelineAsync>g__Awaited|20_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Awaited|17_0(ResourceInvoker invoker, Task task, IDisposable scope)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Awaited|17_0(ResourceInvoker invoker, Task task, IDisposable scope)
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
   at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
   at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
   at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at Ombi.ApiKeyMiddlewear.Invoke(HttpContext context) in /home/runner/work/Ombi/Ombi/src/Ombi/Middleware/ApiKeyMiddlewear.cs:line 58
   at Ombi.ErrorHandlingMiddleware.Invoke(HttpContext context) in /home/runner/work/Ombi/Ombi/src/Ombi/Middleware/ErrorHandlingMiddlewear.cs:line 30
{"message":"API rate limit exceeded for 142.198.127.197. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)","documentation_url":"https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting"}

2023-10-15 19:34:47.593 -04:00 [ERR] StatusCode: BadRequest, Reason: Bad Request, RequestUri: http://127.0.0.1:7879/french/api/v3/movie
2023-10-15 19:34:47.593 -04:00 [ERR] QualityProfile does not exist
2023-10-15 19:34:47.593 -04:00 [WRN] Tried auto sending movie but failed. Message: QualityProfile does not exist
2023-10-15 19:34:47.593 -04:00 [WRN] Tried auto sending movie but failed. Message: QualityProfile does not exist
2023-10-15 19:35:33.292 -04:00 [ERR] StatusCode: BadRequest, Reason: Bad Request, RequestUri: http://127.0.0.1:7879/french/api/v3/movie
@ExtremeFiretop ExtremeFiretop changed the title QualityProfile does not exist QualityProfile does not exist on Radarr-4K Oct 16, 2023
@ExtremeFiretop
Copy link
Contributor Author

With more testing, I found that even though in Ombi on Radarr-4K i selected the profile "Ultra-HD" it requests the profile "SD" in Radarr-4K.

@tidusjar
Copy link
Member

If you are messing around with the profiles in radarr, You need to go into the Radarr settings in ombi and re-select the correct profiles. Ombi keeps a reference to the Profile you have saved originally.

@ExtremeFiretop
Copy link
Contributor Author

ExtremeFiretop commented Oct 16, 2023

If you are messing around with the profiles in radarr, You need to go into the Radarr settings in ombi and re-select the correct profiles. Ombi keeps a reference to the Profile you have saved originally.

As I mentioned, the original Profile selected was "Ultra-HD", and yet Ombi requests "SD" Profile from Radarr. Changing the selection makes no difference.

@ExtremeFiretop
Copy link
Contributor Author

ExtremeFiretop commented Nov 13, 2023

If you are messing around with the profiles in radarr, You need to go into the Radarr settings in ombi and re-select the correct profiles. Ombi keeps a reference to the Profile you have saved originally.

I found the problem.

image

I had set a specific profile for my user at some point in the past, and now that we have Radarr 4K enabled, it had a different profile than my "Regular Radarr" instance.

Checking the SQL database, Radarr profile was set to profile "2" for my user which equates to "SD" on my radarr setup.
There should probably be a new entry for "RadarrQualityProfile4K" where it saves the profile of the 4K instance for specific users, instead of overriding the the 4K profile with the regular Radarr Profile.

@ExtremeFiretop
Copy link
Contributor Author

ExtremeFiretop commented Nov 13, 2023

If you are messing around with the profiles in radarr, You need to go into the Radarr settings in ombi and re-select the correct profiles. Ombi keeps a reference to the Profile you have saved originally.

It seems to me that if you have lets say 2 Profiles on Radarr, and 5 Profiles on Radarr 4k. "RadarQualityProfile" = 2 would be a completely different profile between Radar and Radarr 4K. So User selected profiles should have a selection for Radar 4K, (if you have 4K control enabled.)

@tidusjar
Copy link
Member

tidusjar commented Nov 13, 2023

Looks like I missed that. Nice investigation!!

@ExtremeFiretop
Copy link
Contributor Author

Looks like I missed that. Nice investigation!!

Just happy to finally have a root cause, it was gonna drive me nuts.

Now that I know what caused it I can sleep at night, I cleared out the save and reset it to 0 for my user in the database (default) and all is working as expected again.

The bug is still legit, but at least I have the solution

tidusjar added a commit that referenced this issue Nov 18, 2023
…uality profiles. There are now user quality profiles for 4k profile #5025
@tidusjar
Copy link
Member

This has been fixed in the develop branch! 🎉 🎈

@ExtremeFiretop
Copy link
Contributor Author

This has been fixed in the develop branch! 🎉 🎈

Thank you again!

tidusjar added a commit that referenced this issue Jul 22, 2024
Co-authored-by: tidusjar <[email protected]>
Co-authored-by: sephrat <[email protected]>
Co-authored-by: Conventional Changelog Action <[email protected]>
Co-authored-by: Teifun2 <[email protected]>
Co-authored-by: contrib-readme-bot <[email protected]>
Co-authored-by: dr3amer <[email protected]>
Co-authored-by: echel0n <[email protected]>
Co-authored-by: Marley <[email protected]>
Co-authored-by: Igor Borges <[email protected]>
Co-authored-by: Lucane <[email protected]>
Co-authored-by: mkgeeky <[email protected]>
Co-authored-by: Miguel A Vico Moya <[email protected]>
Co-authored-by: Hadrien <[email protected]>
Co-authored-by: Victor Usoltsev <[email protected]>
Co-authored-by: Wesley King <[email protected]>
Co-authored-by: Lea <[email protected]>
Co-authored-by: snyk-bot <[email protected]>
Co-authored-by: snyk-bot <[email protected]> [skip ci]
Co-authored-by: Jamie <[email protected]>
Co-authored-by: Drew <[email protected]> [skip ci]
Co-authored-by: ryan-c44 <[email protected]>
Co-authored-by: Alexander Russell <[email protected]>
Co-authored-by: Grygon <[email protected]>
Co-authored-by: phildups7 <[email protected]>
Co-authored-by: Avi <[email protected]>
Co-authored-by: Antonin <[email protected]>
Co-authored-by: Drew <[email protected]>
Co-authored-by: Joel Samson <[email protected]>
Co-authored-by: David Benson <[email protected]>
fix: Consistently reset loading flag when requesting movies on discover page. (#4777)
fix(sonarr): 🐛 Fixed an issue where the language list didn't correctly load for power users in the advanced options #4782
fix: Only log error messages from Microsoft (#4787)
fix(notifications): Fixed the Partially TV notifications going to the admin #4797 (#4799)
fix(translations): 🌐 New translations from Crowdin [skip ci] (#4801)
fix(sonarr): 🐛 Sonarr V4 should work now (#4810)
fixed (#4833)
fix(plex-watchlist): Lookup the ID from different sources when Plex doesn't contain the metadata (#4843)
fix(#4847): Invalid Discord request fixed, also fixed an issue where App Only users would not show as logged in on the user management page (#4848)
fix(discover): 🐛 Fixed the default poster not taking into account the base url in some scenarios #4845
fix(wizard): 🐛 Stop access to the wizard when you have already setup ombi (#4866)
fix(plex-oauth): 🐛 Fixed an issue where using OAuth you could log in as a Ombi Local user #4835
fix: Some minor tweaks to the movie info panel (#4883)
fix(sonarr): 🐛 Stop the sonarr version endpoint from breaking when Sonarr is down #4895
fix: Support duplicates in Emby/JF collections (#4902)
fix(discover): Fix denied requests displayed as approved (#4901)
fix: Fix denied movie shown as 'processing request' in details view (#4900)
fix(#4906): 🐛 Fixed an issue with power users and permissions
fix: Cron Validation (#4842)
fix(lidarr): Change monitor to Existing to properly add artist #3597
fix(API): Allow RequestOnBehalf rights if requested from the API (#4919)
Fixes #4610
fix: added media type tag to media type text (#4638)
fix(sickrage): Fixed issue with incorrect handling of SiCKRAGE episode results returned during episode status changes, now expects array of objects from data path if present (#4648)
fix: Missing Poster broken link fix (#4637)
fix: Improve Swagger documentation (#4652)
fix(API): Fix pagination in some edge cases (#4649)
fix(discover): Carousel touch not working when scrolling page and recommendations and similar movie navigation (#4633)
fix: 🐛 Fixed the Request on Behalf of having blanks (#4667)
fix(images): Retry images with a backoff when we get a Too Many requests from TheMovieDb #4685
fix: Landing and Login page improvements (#4690)
fix(discover): 🐛 Created new Image component to handle 429's from TMDB (#4698) and fixed #4635 (#4699)
fix: Override Sonarr V3 Profiles endpoint (#4678)
fix(4K) :4K request fixes (#4702)
fix(translations): 🌐 New translations from Crowdin [skip ci] (#4713)
fix: fixed trakt image not loading when base url present (#4711)
fix: 🐛 Fixed missing externals (#4712)
fix: Log Microsoft warnings to log file (#4723)
fix: Localize recently requested on discover page (#4729)
Fix: Ombi.Api.Lidarr: Remove unused fields from ArtistAdd (#4727)
fix: Fix conflicting property name for Swagger (#4733)
fix: fixed stats controller (#4742)
fix(webhook): Remove added trailing slash from webhook URL #4710
fix(sonarr): 🐛 Cleaned up and removed Sonarr v3 option, sonarr v3 is now the default. This allows us to get ready for the upcoming Sonarr v4 (#4764)
fix: Fixes default image for recently requested items. (#4767)
Fixes build warnings. (#4769)
fix: Reworked the version check (#4719) (#4781)
fix: Unable to Delete Jellyfin Server (#4705) (#4780)
fix: Partially Available prevents further TV requests (#4768) (#4779)
fix(emby): Fix Emby played sync running a full sync during recently added sync (#4932)
Fixes #4947
fix: Fix various styling issues (#4935)
fix(translations): 🌐 New translations from Crowdin [skip ci] (#4926)
fix: upgrade @microsoft/signalr from 6.0.11 to 6.0.16 (#4964) [skip ci]
fix: upgrade primeng from 15.0.0-rc.1 to 15.4.1 (#4962) [skip ci]
fix: src/Ombi.Notifications/Ombi.Notifications.csproj to reduce vulnerabilities (#4969) [skip ci]
fix: upgrade @fortawesome/fontawesome-free from 6.1.2 to 6.4.0 (#4965) [skip ci]
fix: upgrade multiple dependencies with Snyk (#4963) [skip ci]
fix flaky tests (#4970) [skip ci]
fix: More automation tests mainly around the Plex Settings page (#4821)
fix: upgrade cypress-real-events from 1.7.4 to 1.8.1 (#4968) [skip ci]
fix: Remove Angular TSLint (#4973)
fix: upgrade zone.js from 0.11.8 to 0.13.0 (#4975)
fix: upgrade jquery from 3.6.1 to 3.7.0 (#4974)
fix: upgrade multiple dependencies with Snyk (#4961)
fixes (#4978)
fix(user-importer): 🐛 Fixed an issue where the cleanup wouldn't delete users #4812
fix(user-importer): Do not delete the Plex Admin as part of the user Importer cleanup #4870 (#4981)
Fixes #4957
fix: Remove old trending source (#4987)
fix(plex-api): Switch over to the new API to avoid deprecation & save… (#4986)
fix: switch back to the old plex friends API #4989
fix(user-importer): Fixed not importing all correct users #4989
fix: upgrade cypress-real-events from 1.8.1 to 1.9.1 (#5000) [skip ci]
fix: upgrade @microsoft/signalr from 6.0.18 to 6.0.20 (#4999) [skip ci]
fix: upgrade @fortawesome/fontawesome-free from 6.4.0 to 6.4.2 (#5005) [skip ci]
fix: upgrade @types/jquery from 3.5.16 to 3.5.17 (#5011) [skip ci]
fix: src/Ombi/ClientApp/package.json & src/Ombi/ClientApp/yarn.lock to reduce vulnerabilities (#5010) [skip ci]
fix: upgrade @microsoft/signalr from 6.0.21 to 6.0.22 (#5020)
fix: upgrade @types/jquery from 3.5.18 to 3.5.19 (#5022)
fix: upgrade zone.js from 0.13.1 to 0.13.2 (#5019)
fix: upgrade multiple dependencies with Snyk (#5030)
fix: upgrade cypress-real-events from 1.10.0 to 1.10.1 (#5014)
fix: upgrade jquery from 3.7.0 to 3.7.1 (#5015)
Fix: Linkify logo in newsletter (#5036)
fix: upgrade @types/jquery from 3.5.22 to 3.5.23 (#5034) [skip ci]
fix: upgrade @microsoft/signalr from 6.0.22 to 6.0.23 (#5032) [skip ci]
Fix(mutliple servers): attempting to cache the Plex Content in server (#5018)
fix(radarr-4k): 🐛 Fixed an issue when using Radarr 4k with user set quality profiles. There are now user quality profiles for 4k profile #5025
fix: src/Ombi/ClientApp/package.json & src/Ombi/ClientApp/yarn.lock to reduce vulnerabilities (#5072) [skip ci]
fix: src/Ombi/Ombi.csproj to reduce vulnerabilities (#5066) [skip ci]
fix: upgrade @fortawesome/fontawesome-free from 6.4.2 to 6.5.0 (#5053)  [skip ci]
fix: upgrade @types/jquery from 3.5.27 to 3.5.28 (#5049)  [skip ci]
fix: src/Ombi/ClientApp/package.json & src/Ombi/ClientApp/yarn.lock to reduce vulnerabilities (#5040)  [skip ci]
fix: upgrade moment from 2.29.4 to 2.30.1 (#5075) [skip ci]
fix: upgrade multiple dependencies with Snyk (#5073) [skip ci]
fix: upgrade multiple dependencies with Snyk (#5104) [skip ci]
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

No branches or pull requests

2 participants