-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Could not migrate from os10-os12 #1502
Comments
Interesting, it appears that one migration is being executed twice:
I will investigate this. |
It appears that ALL statements are duplicated. I have to investigate if that's an issue with the formatter of the table or a code issue. |
#1502 (comment) will be addressed with: #1504 Regarding your issue, please paste the output of SELECT client_id from hydra_oauth2_logout_request;
SELECT id from client_id; If you have too many rows there please let me know. This will remove all sensitive data so you can run it as-is.
Run this command against a backup:
And print the resulting error message here please. |
Hi @aeneasr, I finished my work at the old place yesterday which means I could not access their database to debug through anymore. However, I like Hydra you have built I'm more than happy to debug this. Give me some time to recreate the situation. |
Thank you @singlewind :) I'll keep this open in the meanwhile :) |
Hi @aeneasr, have you fixed the bug already. I try to recreate the issue today. Looks like it has been fixed. The index has been migrated successfully although it still displays duplicates in the plan. What I will do is keep trying to populate more data before migrations.
|
We haven't fixed anything in that regard, although we fixed the duplicate migration plan issue on master. It's most likely some flakyness with the foreign keys - I think it could easily be resolved by searching for dupe keys and removing the entries as the rows affected are usually not of great value anyways. I think it's ok to not figure out what's going on exactly as it seems like this is a rare occurence - this is the first report of this issue. Would you be ok with closing this? |
Thanks, @aeneasr for all your help. I'm closing now. |
Describe the bug
Run
hydra migrate sql -e --yes
failed in the middleReproducing the bug
Steps to reproduce the behavior:
Server logs
Server configuration
Expected behavior
Can be migrated successfully.
Environment
The text was updated successfully, but these errors were encountered: