starred/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2026-05-04 18:09:12 +03:00
Code Issues 822 Packages Projects Releases 10 Wiki Activity

[PR #14061] [MERGED] Only consider migrations that have key set for migration.xml migration #13821

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/14061
Author: @JPVenson
Created: 5/5/2025
Status: ✅ Merged
Merged: 5/6/2025
Merged by: @crobibero

Base: master ← Head: bugfix/MigrationKeyOnStartup

📝 Commits (3)

  • 948d090 Only consider migrations that have key set for migration.xml migration
  • 3c3f813 Make Key for Migrations an Obsolete parameter
  • 4fcce06 Fix Startup migrations

📊 Changes

27 files changed (+352 additions, -361 deletions)

View changed files

📝 Jellyfin.Server/Migrations/JellyfinMigrationAttribute.cs (+3 -0)
📝 Jellyfin.Server/Migrations/JellyfinMigrationService.cs (+3 -2)
📝 Jellyfin.Server/Migrations/PreStartupRoutines/CreateNetworkConfiguration.cs (+1 -1)
📝 Jellyfin.Server/Migrations/PreStartupRoutines/MigrateEncodingOptions.cs (+1 -1)
📝 Jellyfin.Server/Migrations/PreStartupRoutines/MigrateMusicBrainzTimeout.cs (+1 -1)
📝 Jellyfin.Server/Migrations/PreStartupRoutines/RenameEnableGroupingIntoCollections.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/AddDefaultCastReceivers.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/AddDefaultPluginRepository.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/CreateUserLoggingConfigFile.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/DisableTranscodingThrottling.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/FixAudioData.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/FixPlaylistOwner.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateActivityLogDb.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateAuthenticationDb.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateDisplayPreferencesDb.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateKeyframeData.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs (+1 -11)
📝 Jellyfin.Server/Migrations/Routines/MigrateRatingLevels.cs (+51 -50)
📝 Jellyfin.Server/Migrations/Routines/MigrateUserDb.cs (+165 -166)
📝 Jellyfin.Server/Migrations/Routines/MoveExtractedFiles.cs (+1 -1)

...and 7 more files

📄 Description

MigrateKeyframeData and MigrateRatingLevels has a new key set which is a workaround to reexecute the migration on next run so we get the same result by removing the legacy key.

MigrateLibraryDb, MoveExtractedFiles, MoveTrickplayFiles and RefreshInternalDateModified dont not need a key as they never been executed.

🔄 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/14061 **Author:** [@JPVenson](https://github.com/JPVenson) **Created:** 5/5/2025 **Status:** ✅ Merged **Merged:** 5/6/2025 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `bugfix/MigrationKeyOnStartup` --- ### 📝 Commits (3) - [`948d090`](https://github.com/jellyfin/jellyfin/commit/948d090efc9217c37dfe61f846fddb8dd5f5b799) Only consider migrations that have key set for migration.xml migration - [`3c3f813`](https://github.com/jellyfin/jellyfin/commit/3c3f8133633b895ec16e03673f28ae49e001a33e) Make Key for Migrations an Obsolete parameter - [`4fcce06`](https://github.com/jellyfin/jellyfin/commit/4fcce0631dc1d9c931a020c4b6f7b47bcba88a48) Fix Startup migrations ### 📊 Changes **27 files changed** (+352 additions, -361 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Server/Migrations/JellyfinMigrationAttribute.cs` (+3 -0) 📝 `Jellyfin.Server/Migrations/JellyfinMigrationService.cs` (+3 -2) 📝 `Jellyfin.Server/Migrations/PreStartupRoutines/CreateNetworkConfiguration.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/PreStartupRoutines/MigrateEncodingOptions.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/PreStartupRoutines/MigrateMusicBrainzTimeout.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/PreStartupRoutines/RenameEnableGroupingIntoCollections.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/AddDefaultCastReceivers.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/AddDefaultPluginRepository.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/CreateUserLoggingConfigFile.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/DisableTranscodingThrottling.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/FixAudioData.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/FixPlaylistOwner.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateActivityLogDb.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateAuthenticationDb.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateDisplayPreferencesDb.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateKeyframeData.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs` (+1 -11) 📝 `Jellyfin.Server/Migrations/Routines/MigrateRatingLevels.cs` (+51 -50) 📝 `Jellyfin.Server/Migrations/Routines/MigrateUserDb.cs` (+165 -166) 📝 `Jellyfin.Server/Migrations/Routines/MoveExtractedFiles.cs` (+1 -1) _...and 7 more files_ </details> ### 📄 Description MigrateKeyframeData and MigrateRatingLevels has a new key set which is a workaround to reexecute the migration on next run so we get the same result by removing the legacy key. MigrateLibraryDb, MoveExtractedFiles, MoveTrickplayFiles and RefreshInternalDateModified dont not need a key as they never been executed. --- <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:43 +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#13821
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?