starred/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2026-07-16 05:43:54 +03:00
Code Issues 822 Packages Projects Releases 17 Wiki Activity

[PR #14051] [CLOSED] EFCore/#13049: update database to 10.10.z before EFCore migration is applied #13813

New Issue
Closed
opened 2026-02-07 07:19:37 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-07 07:19:37 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/14051
Author: @TheMelmacian
Created: 5/3/2025
Status: ❌ Closed

Base: master ← Head: efcore/#13049_migrate_database_to_10.10.z

📝 Commits (10+)

  • ec56174 add migration routine to update database to v10.10.z
  • 70c84f9 fix(JellyfinMigrationService): prevent execution of already applied legacy migrations
  • 40a6e51 Update Jellyfin.Server/Migrations/Routines/UpdateDatabaseToTenDotTen.cs
  • 981d6eb apply suggestions from code review
  • b9e2e64 fix error when checking legacy migrations
  • cc57c4a remove obsolete statements to simplify migration routine
  • 662f686 remove UserDatas update scripts
  • 955075f define columns in a list
  • 0aa8304 Integrate branch 'master' into efcore/#13049_migrate_database_to_10.10.z
  • f64f809 Integrate branch 'master' into efcore/#13049_migrate_database_to_10.10.z

📊 Changes

4 files changed (+231 additions, -46 deletions)

View changed files

📝 Jellyfin.Server/Migrations/JellyfinMigrationService.cs (+42 -44)
📝 Jellyfin.Server/Migrations/Routines/MigrateLibraryDbCompatibilityCheck.cs (+1 -1)
➕ Jellyfin.Server/Migrations/Routines/UpdateDatabaseToTenDotTen.cs (+187 -0)
📝 Jellyfin.Server/Program.cs (+1 -1)

📄 Description

Changes

  • added a migration routine that contains the migration code from the old SqliteItemRepository.cs and SqliteUserDataRepository.cs
  • migration of existing migration.xml files is executed regardless of the IsStartupWizardCompleted flag to prevent execution of already applied migrations

Issues
Fixes #13049

Summary by CodeRabbit

  • New Features

    • Added a new database migration routine to update the library database for compatibility with version 10.10.z, ensuring required columns are present and creating backups automatically.

  • Refactor

    • Improved the migration process for legacy migration records, streamlining the detection and application of old migrations for smoother upgrades.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/14051 **Author:** [@TheMelmacian](https://github.com/TheMelmacian) **Created:** 5/3/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `efcore/#13049_migrate_database_to_10.10.z` --- ### 📝 Commits (10+) - [`ec56174`](https://github.com/jellyfin/jellyfin/commit/ec56174e6dc57413f4c5904ee59de723c5a52ce8) add migration routine to update database to v10.10.z - [`70c84f9`](https://github.com/jellyfin/jellyfin/commit/70c84f998e83468823cf498401989d8034e9510e) fix(JellyfinMigrationService): prevent execution of already applied legacy migrations - [`40a6e51`](https://github.com/jellyfin/jellyfin/commit/40a6e5197c646bd104113a2d222f6b885715257e) Update Jellyfin.Server/Migrations/Routines/UpdateDatabaseToTenDotTen.cs - [`981d6eb`](https://github.com/jellyfin/jellyfin/commit/981d6eb15dd6493c7f6e7eeb32af4ab806316397) apply suggestions from code review - [`b9e2e64`](https://github.com/jellyfin/jellyfin/commit/b9e2e6472a9f3e2e4ba257a6f58308ebddaa4257) fix error when checking legacy migrations - [`cc57c4a`](https://github.com/jellyfin/jellyfin/commit/cc57c4a1f3462e3973d36b8e49eb5802f529612a) remove obsolete statements to simplify migration routine - [`662f686`](https://github.com/jellyfin/jellyfin/commit/662f686b004c6ffe4d9292e37dc56db90ad773f8) remove UserDatas update scripts - [`955075f`](https://github.com/jellyfin/jellyfin/commit/955075f2834f5c0f8e2f13eed0e51c0f4c774f0b) define columns in a list - [`0aa8304`](https://github.com/jellyfin/jellyfin/commit/0aa8304ae64635890be7388152841a4be9a7f9ec) Integrate branch 'master' into efcore/#13049_migrate_database_to_10.10.z - [`f64f809`](https://github.com/jellyfin/jellyfin/commit/f64f8094206f696ba0ebe70cda5e5e6ef34055c1) Integrate branch 'master' into efcore/#13049_migrate_database_to_10.10.z ### 📊 Changes **4 files changed** (+231 additions, -46 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Server/Migrations/JellyfinMigrationService.cs` (+42 -44) 📝 `Jellyfin.Server/Migrations/Routines/MigrateLibraryDbCompatibilityCheck.cs` (+1 -1) ➕ `Jellyfin.Server/Migrations/Routines/UpdateDatabaseToTenDotTen.cs` (+187 -0) 📝 `Jellyfin.Server/Program.cs` (+1 -1) </details> ### 📄 Description **Changes** - added a migration routine that contains the migration code from the old `SqliteItemRepository.cs` and `SqliteUserDataRepository.cs` - migration of existing `migration.xml` files is executed regardless of the `IsStartupWizardCompleted` flag to prevent execution of already applied migrations **Issues** Fixes #13049 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Added a new database migration routine to update the library database for compatibility with version 10.10.z, ensuring required columns are present and creating backups automatically. - **Refactor** - Improved the migration process for legacy migration records, streamlining the detection and application of old migrations for smoother upgrades. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 07:19:37 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
EFjellyfin.db
area:database
awaiting-feedback
backend
blocked
breaking change: web api
bug
build
ci
confirmed
discussion needed
dotnet future
downstream
duplicate
enhancement
feature
future
github-actions
good first issue
hdr
help wanted
invalid
investigation
librarydb
live-tv
lyrics
media playback
music
needs testing
nuget
performance
platform
pull-request
Mirrored from GitHub Pull Request
 question
regression
release critical
requires-web
roadmap
security
security
stale
support
syncplay
ui & ux
upstream
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13813
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?