fix: report SQL restore failures and stop subsequent statements #6

Merged

clement merged 1 commit from fix/version-tags-postgres-restore-errors-20260925 into main

2026-09-25 15:52:38 +00:00

clement commented

2026-09-25 15:52:17 +00:00

Owner

Lancer psql sans fichier de démarrage utilisateur et avec ON_ERROR_STOP=1. Une erreur SQL devient un échec de restauration et interrompt le script. Cela ne rend pas la restauration atomique : les instructions exécutées avant l’erreur peuvent déjà avoir modifié la base.

Ajouter un test CI pour PostgreSQL 16, 17 et 18 qui exige un code non nul, l’erreur SQL attendue, la présence du témoin antérieur et l’absence du témoin postérieur. Tous les tests et contrôles de promotion existants sont conservés. Validation locale : reproduction avant correction, refus après correction, puis restauration valide contenant une valeur témoin 42.

Le client sidecar-restore annonçait une réussite après une erreur SQL et exécutait les instructions suivantes : psql ne renvoyait pas de code d’échec pour ces erreurs. Ce comportement a été reproduit dans une base isolée avec une fonction inexistante suivie d’une création de table. Lancer psql sans fichier de démarrage utilisateur et avec ON_ERROR_STOP=1. Une erreur SQL devient un échec de restauration et interrompt le script. Cela ne rend pas la restauration atomique : les instructions exécutées avant l’erreur peuvent déjà avoir modifié la base. Ajouter un test CI pour PostgreSQL 16, 17 et 18 qui exige un code non nul, l’erreur SQL attendue, la présence du témoin antérieur et l’absence du témoin postérieur. Tous les tests et contrôles de promotion existants sont conservés. Validation locale : reproduction avant correction, refus après correction, puis restauration valide contenant une valeur témoin 42.

clement added 1 commit

2026-09-25 15:52:17 +00:00