0 Fork 0

Files

T

ayurishchevandClaude Opus 5.5 5f7bd6588a Состояния «Upgrade FW» и осознанный откат прошивки RouterBOARD

Пункт 18 ревью 2026-09-28 21:32 (docs/changes/027): после отката ROS
плата остаётся на более новой прошивке, а приложение сравнивало версии
на неравенство и показывало откат как обновление.
- devices.fw_state: unknown/update/downgrade/current; has_fw_update и
  фильтры не считают откат обновлением; метка «в ROS: X».
- upgrade_firmware не понижает прошивку; общая запись — _flash_firmware.
- Задача fw_downgrade: проверка → обязательный бэкап → запись →
  перезагрузка; API POST /devices/{id}/firmware/downgrade и
  /batch/fw_downgrade (target_version обязателен).
- UI: «Откатить прошивку…» (строка и группа), общее окно отката с kind.

Тесты: 39 из 39. Стенд: временное недоступное устройство — 422/202,
задачи failed на проверке, бэкапов нет. Реальная запись прошивки при
откате и ручная проверка UI не выполнялись.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

2026-09-29 00:04:51 +03:00

6.6 KiB

Raw Permalink Blame History

Ревью кодовой базы ros_control — 2026-09-28 21:32 MSK

Четвёртое ревью. Состояние на коммит e1f197c. Тесты: 36/36 проходят. Предыдущие ревью: 2026-09-27, 2026-09-28 12:43, 2026-09-28 17:35 (коммит 6ff7f1b).

Итог

Из 19 замечаний закрыто 15, частично — 1 (п. 13), открыто 3 (п. 11, 15, 19) и 1 отложено (п. 18). С предыдущего ревью закрыты п. 12, 14, 16 и подтверждён на практике п. 17.

Изменение	Коммит	Пункты
`018-correctness-consistency`	`123b5ab`	5, 6, 7
`019-performance-scaling`	`ba8ac7b`	8, 9 (частично), 10
`020-custom-select-menus`	`6b5c521`	— (доработка UI по запросу)
`021-security-hardening`	`c407972`	1, 2, 3, 4
`022-async-db-remainder`	`0bda003`	9 (остаток)
`023-ros-downgrade`	`ae826fc`	— (отображение и откат ROS по запросу)
`024-readme-restructure`	`7c830be`	13 (документация, с неточностью)
`025-trusted-proxies-port`	`5367cb5`	14, 16
`026-split-tests`	`e1f197c`	12

Статус замечаний

Закрытые ранее (п. 1–10)

Без изменений: безопасность (1–4, изменение 021), корректность (5–7, 018), производительность (8–10, 019 и 022). Регресс п. 9 контролирует тест-линтер tests/test_architecture.py::test_no_sync_db_calls_in_async_functions.

П. 11–19

№	Замечание	Приоритет	Статус	Подтверждение
11	Нет healthcheck, логирование не настроено	Низкий	❌ Открыт	Нет `HEALTHCHECK` в `Dockerfile`/`docker-compose.yml`, нет `/healthz`; `logging` не настраивается
12	Тесты в одном файле	Низкий	✅ 026	7 модулей по 68–185 строк (всего 1035), `tests/helpers.py`; тела тестов не менялись (AST совпадает), каждый модуль проходит отдельно
13	Копирование БД в режиме WAL	Средний	🟡 Неточность	`README.md:161` советует `sqlite3 <БД> ".backup …"`; в контейнере (`python:3.12-slim`) утилиты `sqlite3` нет, том на хост не смонтирован. Рабочий способ — `sqlite3.Connection.backup` через `docker exec … python -c …`
14	За reverse-proxy блокировка видит IP прокси	Низкий	✅ 025	`app/security.py::client_ip` + `TRUSTED_PROXIES` (по умолчанию пусто — XFF игнорируется); на стенде поддельный `X-Forwarded-For` не повлиял на блокировку и журнал
15	`refresh=1` остаётся в адресе «Бэкапов»	Низкий	❌ Открыт	`app/ui/routes.py:485` — `refresh_url`
16	Порт стенда не параметризован	Низкий	✅ 025	`"${APP_BIND:-0.0.0.0}:${APP_PORT:-8000}:8000"`; стенд на `0.0.0.0:8001` через `APP_PORT` в `.env`, override-файл не нужен
17	Откат ROS не проверен на устройстве	Средний	✅ Подтверждён	См. «Проверка отката на устройстве»
18	Прошивка RouterBOARD после отката остаётся новее	Низкий	✅ 027	Состояния колонки «Upgrade FW» (`unknown`/`update`/`downgrade`/`current`) и осознанный откат FW до версии, встроенной в установленную ROS, по образцу отката ROS (023) — план · итог
19	Тест-линтер видит одну ступень транзитивности	Низкий	❌ Открыт	`tests/test_architecture.py:54–56` — только `level0`/`level1`

Проверка отката на устройстве (п. 17)

Пользователь выполнил реальный откат на 5G-AC-BED через UI (актор ui:tstark). Журнал событий (UTC):

Время	Событие
14:26:06	`job.created` / `job.started` — задача `ros_downgrade`
14:26:07 → 14:26:10	`backup.created` → `backup.done` — бэкап `backups/5G-AC-BED/20260928-142607.{backup,rsc}` до установки
14:26:26	`job.done` — «Откат 7.24.4 → 7.23.7 запущен, устройство перезагрузится»
14:26:31 → 14:29:54	`device.offline` → `device.online` — перезагрузка
14:34:41	задача `ros_update`: «Обновление 7.23.7 → 7.24.4 запущено» — подтверждает, что откат установил 7.23.7
14:35:04 → 14:37:53	перезагрузка, устройство снова на 7.24.4 (stable)

Вывод: /system/package/update/install в RouterOS 7 при версии канала старше установленной выполняет откат; порядок «проверка → бэкап → установка» соблюдён; возврат штатным обновлением работает.

Проверка UI

Ручная проверка интерфейса по 019–026 явно не подтверждена, но фактическая работа пользователя через UI (откат, обновление, бэкапы, смена каналов — события ui:tstark) проходит без ошибок.

6.6 KiB Raw Permalink Blame History Unescape Escape