From a52b0e5555d7f1aa2a9200abfd8ae0391098eeea Mon Sep 17 00:00:00 2001
From: Christian Heusel <christian@heusel.eu>
Date: Mon, 16 Dec 2024 01:36:33 +0100
Subject: [PATCH] mariadb: Add switch for innodb_snapshot_isolation
With the 11.6.2 release mariadb has made snapshot isolation a default
setting which recently caused issues on the AUR and the Forums, where
the respective database engines would fail due to the table changing
since the last time they read it:
PHP Fatal error: Uncaught mysqli_sql_exception: Record has changed since last read in table 'fluxbb_online' in /srv/http/fluxbb/include/dblayer/mysqli_innodb.php:79
Stack trace:
#0 /srv/http/fluxbb/include/dblayer/mysqli_innodb.php(79): mysqli_query()
#1 /srv/http/fluxbb/include/functions.php(485): DBLayer->query()
#2 /srv/http/fluxbb/include/common.php(162): update_users_online()
#3 /srv/http/fluxbb/viewtopic.php(10): require('...')
#4 {main}
We therefore introduce a config switch to restore the old behavior and
apply the changed setting on the two services.
Link: https://mariadb.com/kb/en/mariadb-11-6-2-release-notes/#innodb
Link: https://bbs.archlinux.org/viewtopic.php?id=301802
Link: https://gitlab.archlinux.org/archlinux/aurweb/-/issues/525
Signed-off-by: Christian Heusel <christian@heusel.eu>
---
playbooks/aur.archlinux.org.yml | 2 +-
playbooks/bbs.archlinux.org.yml | 2 +-
roles/mariadb/templates/server.cnf.j2 | 5 +++++
3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/playbooks/aur.archlinux.org.yml b/playbooks/aur.archlinux.org.yml
index a44f93e54..866c3ecbe 100644
--- a/playbooks/aur.archlinux.org.yml
+++ b/playbooks/aur.archlinux.org.yml
@@ -9,7 +9,7 @@
- { role: promtail }
- { role: certbot }
- { role: nginx }
- - { role: mariadb, mariadb_innodb_buffer_pool_size: '1G' }
+ - { role: mariadb, mariadb_innodb_buffer_pool_size: '1G', mariadb_innodb_disable_snapshot_isolation: true }
- { role: sudo }
- { role: valkey }
- { role: uwsgi }
diff --git a/playbooks/bbs.archlinux.org.yml b/playbooks/bbs.archlinux.org.yml
index 300b685d5..4de3c3cfb 100644
--- a/playbooks/bbs.archlinux.org.yml
+++ b/playbooks/bbs.archlinux.org.yml
@@ -7,7 +7,7 @@
- { role: root_ssh }
- { role: certbot }
- { role: nginx }
- - { role: mariadb, mariadb_query_cache_type: '0', mariadb_innodb_buffer_pool_size: '512M' }
+ - { role: mariadb, mariadb_query_cache_type: '0', mariadb_innodb_buffer_pool_size: '512M', mariadb_innodb_disable_snapshot_isolation: true }
- { role: sudo }
- { role: php_fpm, php_extensions: ['apcu', 'iconv', 'intl', 'mysqli'], zend_extensions: ['opcache'] }
- { role: fluxbb }
diff --git a/roles/mariadb/templates/server.cnf.j2 b/roles/mariadb/templates/server.cnf.j2
index 1d5bf186e..fbe139146 100644
--- a/roles/mariadb/templates/server.cnf.j2
+++ b/roles/mariadb/templates/server.cnf.j2
@@ -51,6 +51,11 @@ innodb_thread_concurrency = {{ mariadb_innodb_thread_concurrency }}
innodb_file_per_table
{% endif %}
+{% if mariadb_innodb_disable_snapshot_isolation %}
+# changed to default from OFF to ON with 11.6.2 https://mariadb.com/kb/en/mariadb-11-6-2-release-notes/
+innodb_snapshot_isolation = OFF
+{% endif %}
+
[mysqldump]
quick
max_allowed_packet = 16M
--
GitLab