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

CI: Test with MySQL 5.7 #1012

Closed
wants to merge 1 commit into from
Closed

Conversation

dveeden
Copy link
Collaborator

@dveeden dveeden commented Mar 10, 2025

Closes #775

@dveeden dveeden requested review from atercattus and lance6716 March 10, 2025 07:52
@dveeden dveeden force-pushed the test_mysql57 branch 11 times, most recently from a262f65 to 88bfd98 Compare March 10, 2025 09:37
@dveeden
Copy link
Collaborator Author

dveeden commented Mar 10, 2025

  • It seems to be difficult/impossible to specify container arguments (not options) with service containers
  • Specifying a my.cnf etc. for a service container also seems to be harder that it should be.
  • MySQL 5.7 needs config: server-id=100+log_bin and maybe also TLS config.

@dveeden dveeden closed this Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run tests for mysql 5.x without a github ubuntu-18.04 image
1 participant