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

readinessProbe hits timeout on larger libraries #12

Closed
Travisivart opened this issue Jul 18, 2024 · 1 comment
Closed

readinessProbe hits timeout on larger libraries #12

Travisivart opened this issue Jul 18, 2024 · 1 comment

Comments

@Travisivart
Copy link
Contributor

With larger libraries, the readinessProbe will time out during a FixAudioData Migration

Example from logs:

[08:05:21] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilling audio lyrics data to database.
[08:05:59] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 5000 of 36184 audio records
[08:06:45] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 10000 of 36184 audio records
[08:07:36] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 15000 of 36184 audio records
[08:08:47] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 20000 of 36184 audio records
[08:09:52] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 25000 of 36184 audio records
[08:10:52] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 30000 of 36184 audio records
[08:12:18] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 35000 of 36184 audio records
[08:14:53] [INF] [1] Jellyfin.Server.Migrations.Routines.FixAudioData: Backfilled data for 36184 of 36184 audio records
[08:14:53] [INF] [1] Jellyfin.Server.Migrations.MigrationRunner: Migration 'FixAudioData' applied successfully

It takes nearly 10 minutes for this library of 36184 to be processed. The Probes should be configurable to account for this and give the pod time to finish the Migration.

@Travisivart
Copy link
Contributor Author

Fixed by #11

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

1 participant