Skip to content

fix: MySQL 8.2+ and mysql:8 failing to start - #169

Merged
AaronFeledy merged 6 commits into
mainfrom
fix/mysql-8.4-startup
Oct 15, 2026
Merged

AaronFeledy merged 6 commits into
mainfrom
fix/mysql-8.4-startup

Conversation

@AaronFeledy

@AaronFeledy AaronFeledy commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

MySQL 8.2 removed expire_logs_days, but the recipe's MySQL 8 config still sets it, so mysql:8.2, 8.3 and 8.4 abort on startup with unknown variable 'expire_logs_days=10'. The bare mysql:8 alias (MySQL 8.4.5) was also loading the 5.7 config file and failing on query_cache_limit.

  • Replace expire_logs_days = 10 with binlog_expire_logs_seconds = 864000: same 10 days, supported since MySQL 8.0.1
  • Send mysql:8 to the MySQL 8 config
  • Add examples/acquia-mysql84 on mysql:8 to CI

Verified locally on Lando 3.26.9: lando start with database: mysql:8 boots MySQL 8.4.5 with the MySQL 8 config and binlog_expire_logs_seconds=864000.

Same bug as lando/laravel#114.


Note

Medium Risk
Changes default database config selection and MySQL 8 server settings, which can affect any Acquia app using mysql:8 or 8.x images at startup.

Overview
Fixes MySQL 8.2+ and mysql:8 startup failures in the Acquia recipe by aligning config selection and binlog settings with current MySQL behavior.

In config/mysql8.cnf, expire_logs_days is replaced with binlog_expire_logs_seconds = 864000 (equivalent 10-day retention) so MySQL 8.2+ no longer errors on the removed variable. In builders/acquia-base.js, the MySQL 8 config file is chosen when the version string starts with 8 (not only 8.), so the bare mysql:8 alias gets mysql8.cnf instead of the 5.7 mysql.cnf.

CI adds the acquia-mysql84 Leia example (database: mysql:8) and pins lando/run-leia-action to 1.0.0-beta.4. The changelog documents both fixes.

Reviewed by Cursor Bugbot for commit 011e20b. Bugbot is set up for automated code reviews on this repo. Configure here.

@netlify

netlify Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for lando-acquia ready!

Name Link
🔨 Latest commit 011e20b
🔍 Latest deploy log https://app.netlify.com/projects/lando-acquia/deploys/6ad0bbdc496b284482c915a1
😎 Deploy Preview https://deploy-preview-169--lando-acquia.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 81 (🔴 down 12 from production)
Accessibility: 98 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@AaronFeledy
AaronFeledy merged commit f6574cc into main Oct 15, 2026
32 checks passed
@AaronFeledy
AaronFeledy deleted the fix/mysql-8.4-startup branch October 15, 2026 11:53
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.

1 participant