-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Upgrade PS from 8.0.4 to 8.1.0 - Error upgrading Doctrine schema #33021
Comments
laravel/framework#24711 (comment)
Solution = Use utf8 instead utf8mb4 OR change Ping @PrestaShop/committers |
I got the same error when upgrading from 1.7.7.7 to 8.1 (the same website it's upgraded correctly to 8.0.4). Updating database schema... Errors |
Hello @sallemiines and @Hlavtox, Yes sure, I can make test. But I need more information’s about these two scenarios. Should I go back to version PS 8.0.4 firstly without any changes? A) Change config/database.php B) change name column length to 191 chars Thank you |
Hi @PeNov you can try to change in src/PrestaShopBundle/Entity/AuthorizedApplication.php::56 |
@darfk3d3 @PeNov could you guys provide us with more information about your DB server? You can ask hosting provider, or go to the phpMyAdmin or some other database GUI, so that we know what is the default encoding on your server, not the encoding you have in the tables. This is what I see after login: After getting into the database, you can also see whether you have MyISAM or InnoDb tables. |
@kpodemski |
@mflasquin |
@darfk3d3 all good, thanks |
Hello Krystian @kpodemski , Tables have more type of the collation. For example: utf8mb4_general_ci, utf8_unicode_ci, utf8_general_ci, latin1_swedish_ci MySQL info: Server info: Thank you |
Without testing I guess this new table requires a really big INDEX and is reaching the limits of your MariaDB version: In the link you'll find an explanation: https://dev.mysql.com/doc/refman/8.0/en/innodb-limits.html To check row format:
If you can upgrade MariaDB I think problem will be solved easily since the default ROW FORMAT is DYNAMIC. https://mariadb.com/kb/en/innodb-dynamic-row-format/ As stated in the previous link may this works for you both if you cannot upgrade:
May the docs should reflect this new requirement to upgrade to a greater MariaDB version that allows 3072bytes as stated in the links posted. |
We don't understand... is this a bug? Is this a new requirement? Why is there issue here? |
I asked my hosting about upgrading MariaDB version. They said me that it can be done, but it will be done serverwide and the operation is not reversible, so I'm not sure about this because I have dozen of website of the server, and I'm not sure about MariaDB versions compatibility. |
Im also getting "database" errors at the end of auto-upgrade, doesn't really seem to be database related though PHP Version 8.1.16
|
My response is only what MySQL/MariaDB docs says about that error you're getting, both with 10.1.38, and I wouldn't do an upgrade on a production server neither. On a staging site so you can try to upgrade again and then see the results and take a decision. But PrestaShop may change column definition to solve the issue. |
Apparently we use percona... has there been any reported issues with that DB? |
Percona is also a MySQL distribution but you can wait till @mflasquin release the solution and then upgrade. |
I got same error message as you, did you manage to find anything ? |
Hi @PeNov Thank you for your report, I need to know some things with you : Did your issue seems like this one ? Untitled_.Jul.3.2023.12_08.PM.webmMoreover, did your issue seems like this another issue ; #33041 but at the end, I think this issue is too technical for me, ping @PrestaShop/prestashop-core-developers, can someone please reproduce this issue ? Thanks |
Same problem with same errors |
I update my situation, updating mariadb version from 10.1 to 10.11 solved the problem. I just have a minor issue with back office left menu show names in English instead of Italian, but I don't think it could be something related to mariadb. |
@darkf3d3 that's great! :) regarding the back office menu, update your language pack, that should do it - go to International -> Translations and update Italian language :) |
@kpodemski Thanks, now it's 100% Italian. I didn't know that I need to do that after upgrade. It's an expecter behaviour that it's not done automatically during the upgrade? |
@darkf3d3 I honestly don't know :) I used to do it manually, not only after the upgrade, etc., but regularly just to get fresh translations from time to time :) |
@kpodemski Good to know. I will do this too. Thanks ;) |
For those encountering this bug, from version 8.0.4 -> 8.1.0 and higher with doctrine database:
|
I got the same problem, I can't update from 8.0.5 to 8.1.3, got everytime: "Error during database upgrade. You may need to restore your database." "Error upgrading Doctrine schema" I try to use @mflasquin fix but in 8.0.5 I got no AuthorizedApplication.php file I try also to disable psgdpr module, but got same error. I can't update mysql or mariadb version. What I can do? |
Hi there, |
Prerequisites
Describe the bug and add attachments
Hello,
during the upgrade PS 8.0.4 to PS 8.1.0 these errors are show's:
Error during database upgrade. You may need to restore your database.
Error upgrading Doctrine schema
Log:
log_804_to_810.txt
Seams that PS works as version 8.1.0 now. It is not fully tested.
Are you able suggest me necessary steps?
For example:
A) Roll back to 8.0.4 and wait to new version
B) Repair something in the database (run commands) for correct working of e-shop.
Thank you
Best regards
Petr
Expected behavior
Upgrade without errors.
Steps to reproduce
All modules was upgraded to newest including 1-Click Upgrade module (4.16.0)
Go to 1-Click Upgrade and Upgrade - standard as many times before.
PrestaShop version(s) where the bug happened
8.0.4 to 8.1.0
PHP version(s) where the bug happened
7.4.33
If your bug is related to a module, specify its name and its version
autoupgrade v4.16.0
Your company or customer's name goes here (if applicable).
No response
The text was updated successfully, but these errors were encountered: