Skip to content
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

Update MySQL to 8.0 #4890

Closed
csweichel opened this issue Jul 21, 2021 · 5 comments
Closed

Update MySQL to 8.0 #4890

csweichel opened this issue Jul 21, 2021 · 5 comments
Labels
component: db meta: stale This issue/PR is stale and will be closed soon team: webapp Issue belongs to the WebApp team type: work item This issue pertains to something we have to do where the result is not code or a deployed feature

Comments

@csweichel
Copy link
Contributor

We're still running on MySQL 5.7 which is rather slow on some schema modifications - something that MySQL 8 does much better.

Steps forward

  1. Analyse the upgrade guide to make sure we're not relying on defaults that have changed
  2. update chart dependency to pull in 8.0
  3. if need be, bump the TypeORM mysql dependency
  4. run db-tests
  5. observe server and ws-manager bridge for any DB-related errors
@csweichel csweichel added type: work item This issue pertains to something we have to do where the result is not code or a deployed feature component: db labels Jul 21, 2021
@csweichel
Copy link
Contributor Author

/schedule

@mrsimonemms
Copy link
Contributor

This is details in this SO Post. I'll give a brief overview

MySQL 8 authentication has changed which causes a few issues with NodeJS as they've changed the plugin from mysql_native_password to caching_sha2_password. None of the MySQL plugins for NodeJS support this new authentication mechanism.

There's a couple of workarounds:

  • if you have access to the run command (eg, if it's in a Docker container), append the command with --default-authentication-plugin=mysql_native_password
  • if it's a cloud instance, run ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password BY 'MyNewPass';

Although I've never investigated it, these shouldn't affect the speed of the schema modifications.

@csweichel csweichel added the team: webapp Issue belongs to the WebApp team label Aug 24, 2021
@stale
Copy link

stale bot commented Nov 22, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Nov 22, 2021
@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Nov 23, 2021
@JanKoehnlein JanKoehnlein added the meta: stale This issue/PR is stale and will be closed soon label Nov 23, 2021
@JanKoehnlein
Copy link
Contributor

This issue has been set to scheduled by our automation because the label "meta:stale" was added. Unscheduling

@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Nov 23, 2021
@stale
Copy link

stale bot commented Feb 21, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Feb 21, 2022
@stale stale bot closed this as completed Mar 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: db meta: stale This issue/PR is stale and will be closed soon team: webapp Issue belongs to the WebApp team type: work item This issue pertains to something we have to do where the result is not code or a deployed feature
Projects
None yet
Development

No branches or pull requests

4 participants