Update dev tooling to latest MySQL, PostgreSQL#648
Merged
Conversation
stephenfin
commented
Jun 6, 2026
stephenfin
commented
Member
- CI: Update to latest stable MySQL, PostgreSQL
- Fix compat with latest patchwork/pyenv image
stephenfin
force-pushed
the
database-update
branch
5 times, most recently
from
8f60979 to
cb16019
Compare
We switched to the Ubuntu 24.04 base image some time back. This includes an ubuntu user that conflicts with the patchwork user we were creating. Simplify this by using the ubuntu user instead. Signed-off-by: Stephen Finucane <stephen@that.guru>
Signed-off-by: Stephen Finucane <stephen@that.guru>
These are also compatible with Debian. Signed-off-by: Stephen Finucane <stephen@that.guru>
MySQL 9.7 changed the default gtid_mode from OFF to ON (since 9.5). When the MySQL Docker container initialises, the init scripts run with GTID=ON, leaving non-empty GTID_EXECUTED in the binary log even though the final server starts with --gtid-mode=OFF. This causes two problems: 1. mysqldump always includes SET @@GLOBAL.GTID_PURGED when cloning test databases for parallel tests. Restoring this requires SYSTEM_VARIABLES_ADMIN, which was not granted. 2. mysqldump requires PROCESS to dump tablespace information. While PROCESS was being granted, SYSTEM_VARIABLES_ADMIN was missing from the grants, causing clone failures (exit code 5) for Django 6.0's stricter parallel test runner. Fix both issues by: - Running RESET BINARY LOGS AND GTIDS before granting privileges, which clears the GTID state left by the init phase so mysqldump no longer emits GTID_PURGED statements. - Adding SYSTEM_VARIABLES_ADMIN to the privilege grants so that if any GTID state is present (e.g. in future MySQL versions), the restore can set GTID_PURGED. Signed-off-by: Stephen Finucane <stephen@that.guru> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
stephenfin
force-pushed
the
database-update
branch
from
cb16019 to
f1d16ab
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.