v1.9.18: fix OTA - create_db.py manquant dans update scripts

Les MAJ OTA ne lancent pas create_db.py, donc toute nouvelle table
ajoutée par une release (ex: data_S88) reste inexistante en base.
Les timers tournent, le script s'exécute, mais l'INSERT échoue
silencieusement avec 'no such table' — capturé par le try/except,
exit 0, systemd voit success.

Symptôme observé sur les capteurs avec S88 activé: 'Get Data' marche
(live read), mais 'Mesures CO2 (Senseair S88)' montre 'Aucune donnée
disponible dans cette table'.

Fix: create_db.py est appelé en step 2 juste avant set_config.py
dans les deux scripts d'update (git pull et upload fichier).
Idempotent (CREATE IF NOT EXISTS + ALTER ADD COLUMN in try/except).

Self-bootstrap: après cette OTA, tous les capteurs auront toutes
les tables, y compris celles introduites dans des releases passées.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
PaulVua
2026-06-01 17:03:07 +02:00
parent 0f94fda0ba
commit 4e5e1a8144
4 changed files with 30 additions and 5 deletions

View File

@@ -62,9 +62,16 @@ if [ -f "/var/www/nebuleair_pro_4g/VERSION" ]; then
print_status "Firmware version: $(cat /var/www/nebuleair_pro_4g/VERSION)"
fi
# Step 2: Update database configuration
# Step 2: Update database (schema migration + config keys)
# create_db.py is idempotent (CREATE TABLE IF NOT EXISTS + ALTER TABLE ADD COLUMN
# wrapped in try/except). Required to add tables introduced after the sensor was
# initially provisioned (e.g. data_S88, data_NOISE.noise_status, ...).
# Without this step, OTA updates that add a new sensor table leave the timer running
# but every write fails with "no such table".
print_status ""
print_status "Step 2: Updating database configuration..."
print_status "Step 2: Updating database (schema migration + config keys)..."
/usr/bin/python3 /var/www/nebuleair_pro_4g/sqlite/create_db.py
check_status "Database schema migration"
/usr/bin/python3 /var/www/nebuleair_pro_4g/sqlite/set_config.py
check_status "Database configuration update"