From 4bf188eecedec39064e3b1d47c32b000e842abfd Mon Sep 17 00:00:00 2001
From: "marco.locatelli@steamware.net" <marco.locatelli@steamware.net>
Date: Fri, 27 Mar 2026 13:26:31 +0100
Subject: [PATCH] aggiunta commenti backup logic

---
 .../mariadb-backup-logic-per-database.sh      | 77 ++++++++++++++++---
 1 file changed, 66 insertions(+), 11 deletions(-)

diff --git a/mariadb-backup/mariadb-backup-logic-per-database.sh b/mariadb-backup/mariadb-backup-logic-per-database.sh
index 223e488..956bd8f 100644
--- a/mariadb-backup/mariadb-backup-logic-per-database.sh
+++ b/mariadb-backup/mariadb-backup-logic-per-database.sh
@@ -1,31 +1,57 @@
 #!/bin/bash
+# Backup logico per-database con retry, sicuro per cluster Galera
+# Autore: Marco + Copilot
+# Obiettivo: dump affidabile anche su nodi molto attivi
+
 set -euo pipefail
 
+# Aumenta i file descriptor disponibili (mysqldump può aprirne molti)
 ulimit -n 65536
 
+# Directory base dei backup
 BACKUP_BASE=/var/backups/tscale01
+
+# Prefisso per distinguere i backup logici di questo nodo
 PREFIX=eqn-bck-logic-
+
+# Timestamp per la directory del backup
 TIMESTAMP=$(date +%Y%m%d-%H%M%S)
+
+# Directory finale del backup
 TARGET="$BACKUP_BASE/${PREFIX}${TIMESTAMP}"
 
+# File di log generale
 LOGFILE=/var/log/mariadb-backup.log
+
+# Lockfile per evitare esecuzioni concorrenti
 LOCKFILE=/var/lock/mariadb-backup.lock
 
+# Nome del server per i log (utile in ambienti multi-nodo)
 SERVER_NAME=$(hostname -s)
 
-# binari e opzioni (root senza password)
+# Binari necessari
 MYSQL_BIN=$(command -v mariadb || true)
 MYSQLDUMP_BIN=$(command -v mariadb-dump || true)
+
+# Opzioni dump ottimizzate per Galera (no lock, snapshot consistente)
 MYSQLDUMP_OPTS="--user=root --single-transaction --quick --skip-lock-tables"
 
-# logfile
+# Numero massimo di retry per ogni database (deadlock = retry)
+MAX_RETRIES=10
+
+# Secondi di attesa tra un retry e l'altro
+RETRY_SLEEP=30
+
+# Assicura che il logfile esista e abbia permessi sicuri
 touch "$LOGFILE"
 chown root:root "$LOGFILE"
 chmod 600 "$LOGFILE"
 
 (
+  # FLOCK: evita che due backup partano insieme
   flock -n 9 || { echo "[$(date '+%F %T')] [$SERVER_NAME] SKIP: another backup is running" >> "$LOGFILE"; exit 0; }
 
+  # Controllo binari
   if [ -z "$MYSQL_BIN" ]; then
     echo "[$(date '+%F %T')] [$SERVER_NAME] ERROR: mariadb client not found in PATH" >> "$LOGFILE"
     exit 1
@@ -36,16 +62,18 @@ chmod 600 "$LOGFILE"
     exit 1
   fi
 
+  # Crea directory del backup
   mkdir -p "$TARGET"
   chown mysql:mysql "$TARGET"
   chmod 750 "$TARGET"
 
+  # File temporaneo per catturare errori del dump
   TMPLOG=$(mktemp /tmp/mariadb-backup-logic.XXXXXX)
 
   echo "---------------------" >> "$LOGFILE"
   echo "[$(date '+%F %T')] [$SERVER_NAME] START logic backup (per-database) $TARGET" >> "$LOGFILE"
 
-  # elenco database (escludo schemi di sistema)
+  # Elenco database utente (escludiamo schemi di sistema)
   DB_LIST=$($MYSQL_BIN -N -e "SHOW DATABASES" | grep -vE '^(information_schema|performance_schema|mysql|sys)$' || true)
 
   if [ -z "$DB_LIST" ]; then
@@ -55,22 +83,48 @@ chmod 600 "$LOGFILE"
   FAILED=0
   FAILED_DBS=""
 
+  # Loop su ogni database
   for db in $DB_LIST; do
     DUMPFILE="$TARGET/${db}.sql"
     echo "[$(date '+%F %T')] [$SERVER_NAME] START dump database '$db' -> $DUMPFILE" >> "$LOGFILE"
 
-    if $MYSQLDUMP_BIN $MYSQLDUMP_OPTS "$db" >"$DUMPFILE" 2>"$TMPLOG"; then
-      echo "[$(date '+%F %T')] [$SERVER_NAME] END dump database '$db'" >> "$LOGFILE"
-    else
-      echo "[$(date '+%F %T')] [$SERVER_NAME] ERROR dumping database '$db'. See $TMPLOG" >> "$LOGFILE"
-      tail -n 100 "$TMPLOG" >> "$LOGFILE"
+    attempt=1
+    success=0
+
+    # Retry intelligente in caso di deadlock (errore 1213)
+    while [ "$attempt" -le "$MAX_RETRIES" ]; do
+      : >"$TMPLOG"   # pulisce il file temporaneo
+
+      if $MYSQLDUMP_BIN $MYSQLDUMP_OPTS "$db" >"$DUMPFILE" 2>"$TMPLOG"; then
+        echo "[$(date '+%F %T')] [$SERVER_NAME] END dump database '$db' (attempt $attempt)" >> "$LOGFILE"
+        success=1
+        break
+      else
+        echo "[$(date '+%F %T')] [$SERVER_NAME] ERROR dumping database '$db' (attempt $attempt). See $TMPLOG" >> "$LOGFILE"
+        tail -n 50 "$TMPLOG" >> "$LOGFILE"
+
+        # Se è un deadlock, riproviamo
+        if grep -q "Error 1213" "$TMPLOG"; then
+          echo "[$(date '+%F %T')] [$SERVER_NAME] RETRY for database '$db' in ${RETRY_SLEEP}s due to deadlock (1213)" >> "$LOGFILE"
+          sleep "$RETRY_SLEEP"
+        else
+          # Errori non recuperabili → stop retry
+          echo "[$(date '+%F %T')] [$SERVER_NAME] NON-RETRYABLE error for database '$db'" >> "$LOGFILE"
+          break
+        fi
+      fi
+
+      attempt=$((attempt + 1))
+    done
+
+    # Se dopo i retry non è andata → segna come fallito
+    if [ "$success" -ne 1 ]; then
       FAILED=1
       FAILED_DBS="$FAILED_DBS $db"
-      # continuo con i successivi
     fi
   done
 
-  # misura dimensioni complessive
+  # Calcolo dimensioni del backup
   size_apparent_bytes=$(find "$TARGET" -type f -printf '%s\n' 2>/dev/null | awk '{s+=$1} END{print s+0}')
   size_apparent_human=$(numfmt --to=iec --suffix=B "$size_apparent_bytes" 2>/dev/null || echo "${size_apparent_bytes}B")
 
@@ -80,6 +134,7 @@ chmod 600 "$LOGFILE"
   echo "[$(date '+%F %T')] [$SERVER_NAME] SIZE apparent: $size_apparent_human ($size_apparent_bytes bytes) for $TARGET" >> "$LOGFILE"
   echo "[$(date '+%F %T')] [$SERVER_NAME] SIZE on-disk: $size_disk_human ($size_disk_bytes bytes) for $TARGET" >> "$LOGFILE"
 
+  # Risultato finale
   if [ "$FAILED" -eq 0 ]; then
     echo "[$(date '+%F %T')] [$SERVER_NAME] RESULT: OK, all databases dumped successfully" >> "$LOGFILE"
   else
@@ -88,7 +143,7 @@ chmod 600 "$LOGFILE"
 
   rm -f "$TMPLOG"
 
-  # rotazione: mantieni 7 giorni per i logic
+  # Rotazione: elimina backup più vecchi di 7 giorni
   TO_DELETE=$(find "$BACKUP_BASE" -maxdepth 1 -type d -name "${PREFIX}*" -mtime +7 -print 2>/dev/null || true)
 
   if [ -n "$TO_DELETE" ]; then