-
Notifications
You must be signed in to change notification settings - Fork 135
Fix Debian 13 compatability #272
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
base: master
Are you sure you want to change the base?
Conversation
|
In theory this deb822 style of configuring should also work with previous versions of Debian, but I have not tested that at all. Until someone had the time to fix Molecule, maybe someone can verify my PR? |
| mariadb_debian_repo_keyserver: "keyserver.ubuntu.com" | ||
| mariadb_debian_repo_pin: "nyc2.mirrors.digitalocean.com" | ||
| mariadb_debian_repo_pin_priority: 600 | ||
| mariadb_debian_repo: "https://deb.mariadb.org/{{ mariadb_version }}/{{ ansible_distribution|lower }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coming from https://mariadb.org/download/?t=repo-config&d=Debian+13+%22Trixie%22; deb.mariadb.org is a dynamic mirror
|
Debian 13 Molecule tests were failing: That's because for Debian 13 Trixie MariaDB's repositories only offer 12.0 or newer. |
After trying to run this playbook on a fresh Debian 13 Trixie VM I debugged step-by-step what wasn't working. Essentially this includes two things:
Description
For 1:
software-properties-commonis not available for Debian 13 anymore, hence breaking the repository setup (compare https://www.reddit.com/r/debian/comments/1n9aakj/why_were_the_softwareproperties_tools_removed/ and dokku/dokku#7877 for example). This PR replaces legacy repository setup and key management with the newer deb822_repository module and removes unnecessary variables, templates, and pre-requisite packages that I assumed not to be necessary anymore.For 2: Updates cluster node address generation in configuration templates to reference dynamic groups instead of static lists, making cluster configuration more flexible and maintainable (
templates/etc/my.cnf.d/server.cnf.j2,templates/etc/my.cnf.d/server.cnf.temp.j2,templates/etc/mysql/conf.d/galera.cnf.j2,templates/etc/mysql/conf.d/galera.cnf.temp.j2)Related Issue
Closes #265
Types of changes
Checklist: