mirror of
https://github.com/koel/koel.git
synced 2026-04-26 01:06:00 +03:00
[GH-ISSUE #1778] Database migration fails on v7.0.0 #980
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#980
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
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?
Originally created by @Mark-Shternberg on GitHub (Jul 11, 2024).
Original GitHub issue: https://github.com/koel/koel/issues/1778
Describe the bug
Updating from 6.12.1 to 7.0.0 fails during database migration with:
Environment
database/migrations/2024_01_03_104241_support_multi_tenant.phpfrom #1774Additional context
Crash logs from laravel.log:
@muellpanda commented on GitHub (Jul 11, 2024):
Fails here too, but different error:
@Mark-Shternberg commented on GitHub (Jul 11, 2024):
My problem solved by rollback from backup and repeat. But I get same problem as above:
@phanan commented on GitHub (Jul 11, 2024):
@muellpanda Please create a separate ticket. I can't support two issues at the same time.
@Mark-Shternberg I don't see how the table can "not exist." Please make sure you run the migration from a clean state (i.e. not as a retry after failure).
@Mark-Shternberg commented on GitHub (Jul 11, 2024):
Migration was started from clean state (to be sure i repeated steps). My steps:
After this steps error occurs
@phanan commented on GitHub (Jul 11, 2024):
Oh ok I get it! Let me try something real quick.
@phanan commented on GitHub (Jul 11, 2024):
@Mark-Shternberg In this file
/database/migrations/2024_01_16_223632_add_timestamps_and_user_id_into_playlist_song_table.phpcan you try replacing line 19 with:And try rerunning the migration?
@Mark-Shternberg commented on GitHub (Jul 11, 2024):
@phanan Now another error:
@phanan commented on GitHub (Jul 11, 2024):
@Mark-Shternberg At least we're getting somewhere ;D For this, can you try removing the clause from line 16 to line 27 in
database/migrations/2024_01_27_171649_add_position_into_playlists_table.php?@Mark-Shternberg commented on GitHub (Jul 11, 2024):
@phanan Great! Thanks a lot! Migration done and server works. I hope this doesn't affect performance
@phanan commented on GitHub (Jul 11, 2024):
Thanks for aiding with the debug and sorry for the trouble. There are just so many wild cards; I couldn't cater to all of them.