Skip to content

CI: Test with MySQL 5.7 #1012

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

CI: Test with MySQL 5.7 #1012

wants to merge 1 commit into from

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

Reviewers

@atercattus atercattus Awaiting requested review from atercattus

@lance6716 lance6716 Awaiting requested review from lance6716

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

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

@dveeden