search

tl-its-umich-edu / canvas-course-manager-next

Canvas Course Manager Next: A redesign of the existing CCM application. It extends Canvas features, makes cumbersome features easier to use, and adds new features.
8 stars 9 forks source link

#388 Upgrading to MySQL8 #389

Closed pushyamig closed 2 years ago

pushyamig commented 2 years ago

fixes #388

Testing process:

  1. Able to launch the Application both in the Development and Prod mode
  2. Able to connect to Dbeaver with MySQL driver. Screen Shot 2022-07-01 at 11 51 57 AM
pushyamig commented 2 years ago

@zqian This brings a question that we are using older version of mysql v5 and I don't know in future if they will update these older versions with newer technology changes.

Should be consider upgrading to latest version of mysql?

jonespm commented 2 years ago

I recommended on MyLA to try mysql:8 to see if that works. It looks like the plan is we'll be upgrading our database to that this summer so should start testing on that locally.

zqian commented 2 years ago

FYI, We have scheduled the MySQL 8 update for dev instances next week.

pushyamig commented 2 years ago

Yeah, I saw the discussion on slack. I am going to wait until how the migration goes and update the PR as needed.

jonespm commented 2 years ago

I would make changes here similar to what we did for MyLA. https://github.com/tl-its-umich-edu/my-learning-analytics/commit/c104d4b0656ca9fe918bfb9f9bd072b3b3f8e230

Switch to MySQL 8 Add the --socket option (may or may not be needed)

Add

    environment:
      - MYSQL_ROOT_HOST="0.0.0.0"

You probably don't have the delegated/cached in this project so just the MySQL changes.

pushyamig commented 2 years ago

@ssciolla I request your approval, just to make sure that it run on non-Apple chip machine. But this criteria might not be accurate with upgrade to mysql8. But I want to make sure it runs for you locally without any problem.

ssciolla commented 2 years ago

@ssciolla I request your approval, just to make sure that it run on non-Apple chip machine. But this criteria might not be accurate with upgrade to mysql8. But I want to make sure it runs for you locally without any problem.

@pushyamig, testing now