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

Switch mysql-server in docker-compose.yml to just mysql #1446

Closed
jonespm opened this issue Oct 6, 2022 · 1 comment · Fixed by #1455
Closed

Switch mysql-server in docker-compose.yml to just mysql #1446

jonespm opened this issue Oct 6, 2022 · 1 comment · Fixed by #1455
Assignees
Labels

Comments

@jonespm
Copy link
Member

jonespm commented Oct 6, 2022

All of our other projects are just using

mysql:8.0, which is a link for mysql:8.0-oracle. We should change over MyLA to be consistent with the other projects. I don't think it will be much effort but might require removing the local databases.

image: mysql/mysql-server:8.0

@jonespm jonespm changed the title Switch mysql-server to regular mysql Switch mysql-server in docker-compose.yml to just mysql Oct 6, 2022
@jonespm jonespm added this to To do in MyLA-2022.02.01 via automation Nov 3, 2022
@jonespm jonespm moved this from To do to In progress in MyLA-2022.02.01 Nov 9, 2022
jonespm added a commit to jonespm/student-dashboard-django that referenced this issue Nov 9, 2022
MyLA-2022.02.01 automation moved this from In progress to Review/QA Nov 15, 2022
@jennlove-um jennlove-um moved this from Review/QA to Review/QA - DEV in MyLA-2022.02.01 Nov 28, 2022
@ssciolla
Copy link
Contributor

ssciolla commented Dec 8, 2022

Not seeing any problems with the changes here in local development. Moving to Done.

@ssciolla ssciolla moved this from Review/QA - DEV to Done in MyLA-2022.02.01 Dec 8, 2022
@ssciolla ssciolla removed their assignment Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants