notplantsandClaude Fable 5.1
9a80002b37
continuous-integration/drone/push Build is failing
`scripts/lint.sh --fix` from the pinned lint devshell: 90 Python files reformatted (ruff format, mechanical) and one C420 (dict comprehension → dict.fromkeys) in tests/unit/test_f211_sso_skip.py. The push self-test had been failing at the lint stage since build 1313 (2026-08-31) on exactly these files; nothing else changed. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FqkQq3CDmFWcQ7u1LzoyRz
34 lines
1.8 KiB
Python
34 lines
1.8 KiB
Python
"""ghost — RESTORE overlay (Phase 1e HC3 / Phase 2 P4): data-integrity, assertion-only + additive.
|
|
|
|
ops.pre_restore dropped the ci_marker table (diverge from the backup); the orchestrator restored
|
|
once. This overlay ADDS: the restored DB carries the pre-mutation 'original' marker — proving the
|
|
seeded data actually survived backup→restore, not just that the service came back up.
|
|
|
|
NOTE (expected RED until a recipe-PR lands): the current ghost recipe backs the DB up as a LOGICAL
|
|
mysqldump (`--tab` → SQL files under /var/lib/mysql-files/) but ships **no `backupbot.restore.*`
|
|
hook**, and the actual mysql data volume is NOT itself backupbot-labelled. So a file-level restore
|
|
puts the dump files back on disk but never reimports them into the running MySQL → the dropped marker
|
|
does not return. This is the SAME defect class cc-ci already caught + fixed in immich and
|
|
mattermost-lts (pg_dump backup with no reimport-on-restore). If this test goes RED, the fix is a
|
|
recipe-PR adding a restore post-hook that reimports the dump (terminate/recreate `ghost` DB +
|
|
`mysql ghost < dump` / `mysqlimport`), mirroring the mattermost-lts#1 / immich#1 pattern.
|
|
"""
|
|
|
|
import os
|
|
import sys
|
|
|
|
sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "..", "runner"))
|
|
from harness import lifecycle # noqa: E402
|
|
|
|
|
|
def _mysql(domain, sql):
|
|
cmd = f'MYSQL_PWD="$(cat /run/secrets/db_password)" mysql -u root -N -s ghost -e "{sql}"'
|
|
return lifecycle.exec_in_app(domain, ["sh", "-c", cmd], service="db").strip()
|
|
|
|
|
|
def test_restore_returns_state(live_app):
|
|
assert _mysql(live_app, "SELECT v FROM ci_marker;") == "original", (
|
|
"restore did not return the pre-mutation ghost MySQL marker (data-integrity failure — "
|
|
"recipe likely lacks a mysqldump-reimport restore hook; see module docstring)"
|
|
)
|