Skip to content

docs: modernize MySQL and MariaDB configuration guidance - #15992

Open
melroy89 wants to merge 4 commits into
nextcloud:masterfrom
melroy89:patch-1
Open

melroy89 wants to merge 4 commits into
nextcloud:masterfrom
melroy89:patch-1

Conversation

@melroy89

@melroy89 melroy89 commented Oct 31, 2026 •

Copy link
Copy Markdown

Remove obsolete database configuration options and update the PHP connection guidance.

  • Retain innodb_buffer_pool_instances as a commented-out option with an inline explanation: it is MySQL-only and was removed in MariaDB 10.6. MySQL 8.4 already defaults to one instance for this example's 128M buffer pool.
  • Remove the obsolete innodb_large_prefix and innodb_file_format options.
  • Remove MySQL-incompatible query-cache options from the shared example.
  • Replace the removed PHP mysql.* extension settings with PDO_MYSQL socket guidance. Include a brief Ubuntu PHP 8.5-FPM example using a separate 99-nextcloud.ini override file, the reload command, and separate CLI configuration guidance. Only override an incorrect default socket path.
  • Clarify that some configuration options and include paths depend on the database engine and installation, and correct the MariaDB manual link label.

The repeated obsolete options in config_sample_php_parameters.rst need a separate upstream fix in Nextcloud Server's config.sample.php, since that documentation is generated.

Once merged, please also merge the other PR (in the other repo): nextcloud/server#66121

AI assistance: OpenAI Codex (GPT-6) assisted with the review, documentation changes, local validation, and PR update.

☑️ Resolves

Fix #4758

🖼️ Screenshots

Before:

Previous database configuration guidance

After (database configuration with the inline MySQL-only comment):

Updated database settings

After (compatibility note, PDO socket override, and PHP-FPM/CLI configuration guidance):

Updated database configuration guidance

✅ Checklist

  • I have built the documentation locally and reviewed the output
  • Screenshots are included for visual changes
  • I have not moved or renamed pages (or added a redirect if I did)
  • I have run codespell or similar and addressed any spelling issues

Validation: make -C admin_manual html passed with warnings treated as errors; sphinx-lint, codespell, and git diff --check passed. The rendered page was reviewed in Chrome. Database startup compatibility was not tested.

Removed the innodb_buffer_pool_instances setting since its removed in MariaDB/MySQL. For quite some time now: https://mariadb.com/docs/release-notes/community-server/10.6/10.6.0#innodb-variables

Signed-off-by: Melroy van den Berg <melroy@melroy.org>
@melroy89 melroy89 changed the title Remove innodb_buffer_pool_instances from config docs: modernize MySQL and MariaDB configuration guidance Oct 31, 2026
Remove obsolete InnoDB and query cache options, replace legacy PHP MySQL settings with PDO socket guidance, and clarify engine-specific configuration.

Assisted-by: Codex:GPT-6
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
Assisted-by: Codex:GPT-6
Signed-off-by: Melroy van den Berg <melroy@melroy.org>
Assisted-by: Codex:GPT-6
Signed-off-by: Melroy van den Berg <melroy@melroy.org>

This branch has not been deployed

No deployments
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.

The suggested configuration for PHP MySQL driver is wrong due to MySQL (Original) being removed since PHP 7.0

1 participant