Files
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

221 lines
9.4 KiB
Python

import re
from dataclasses import dataclass
from app import ids, security
from app.config import get_settings
from app.db import session_scope
from app.models import Device, Group
from app.ros.client import RosClient
from app.ros.operations import version_newer
from app.services import events
NAME_RE = re.compile(r"^[A-Za-z0-9._-]{1,64}$") # имя попадает в ключи S3 — только безопасные символы
@dataclass
class Conn:
id: str
name: str
host: str
port: int
username: str
password: str
verify_tls: bool
use_tls: bool
def validate_name(name: str) -> None:
if not NAME_RE.match(name):
raise ValueError("Имя: латиница, цифры, '.', '_', '-' (до 64 символов)")
def list_devices() -> list[Device]:
with session_scope() as s:
return list(s.query(Device).order_by(Device.name))
def get_device(device_id: str) -> Device:
ids.check(device_id, "dev")
with session_scope() as s:
d = s.get(Device, device_id)
if d is None:
raise LookupError(f"Устройство {device_id} не найдено")
return d
def _check_group(s, group_id: str | None) -> None:
if group_id is not None:
ids.check(group_id, "grp")
if group_id is not None and s.get(Group, group_id) is None:
raise LookupError(f"Группа {group_id} не найдена")
NOTE_MAX = 500
def _clean_note(note: str | None) -> str | None:
note = (note or "").strip()
if len(note) > NOTE_MAX:
raise ValueError(f"Примечание: не более {NOTE_MAX} символов")
return note or None
def _resolve_group(s, group_id: str | None, new_group: str | None) -> str | None:
"""Группа устройства. new_group создаёт группу (или берёт существующую с таким именем)
в той же транзакции, что и устройство: при ошибке не остаётся лишней группы."""
name = (new_group or "").strip()
if name:
if len(name) > 64:
raise ValueError("Название группы: от 1 до 64 символов")
g = s.query(Group).filter_by(name=name).first()
if g is None:
g = Group(name=name)
s.add(g)
s.flush()
events.record("group.created", "group", g.id, f"Группа «{name}» создана", data={"name": name}, s=s)
return g.id
_check_group(s, group_id)
return group_id
def create_device(name, host, port, username, password, verify_tls=False, use_tls=True,
group_id: str | None = None, new_group: str | None = None, note: str | None = None) -> Device:
validate_name(name)
note = _clean_note(note)
with session_scope() as s:
group_id = _resolve_group(s, group_id, new_group)
if s.query(Device).filter_by(name=name).first():
raise ValueError(f"Устройство с именем {name} уже существует")
d = Device(name=name, host=host, port=port, username=username,
password_enc=security.encrypt(password), verify_tls=verify_tls, use_tls=use_tls,
group_id=group_id, note=note)
s.add(d)
s.flush()
events.record("device.created", "device", d.id, f"Устройство {name} добавлено", device_id=d.id,
data={"name": name, "host": host, "port": port, "group_id": group_id}, s=s)
return d
def update_device(device_id: str, **fields) -> Device:
ids.check(device_id, "dev")
with session_scope() as s:
d = s.get(Device, device_id)
if d is None:
raise LookupError(f"Устройство {device_id} не найдено")
if fields.get("name") and fields["name"] != d.name: # имя — часть ключей бэкапов в S3
raise ValueError("Имя устройства нельзя изменить")
changed = []
for k in ("host", "port", "username", "verify_tls", "use_tls"):
if fields.get(k) is not None and getattr(d, k) != fields[k]:
setattr(d, k, fields[k])
changed.append(k)
if "group_id" in fields or fields.get("new_group"): # group_id=None — открепить от группы
gid = _resolve_group(s, fields.get("group_id"), fields.get("new_group"))
if gid != d.group_id:
d.group_id = gid
changed.append("group_id")
if "note" in fields: # пустое примечание — очистить
note = _clean_note(fields["note"])
if note != d.note:
d.note = note
changed.append("note")
if fields.get("password"): # пустой пароль = не менять
d.password_enc = security.encrypt(fields["password"])
changed.append("password") # только факт смены, значение в журнал не пишется
if changed:
events.record("device.updated", "device", d.id, f"Устройство {d.name} изменено: {', '.join(changed)}",
device_id=d.id, data={"fields": changed}, s=s)
return d
def delete_device(device_id: str) -> None:
ids.check(device_id, "dev")
with session_scope() as s:
d = s.get(Device, device_id)
if d is None:
raise LookupError(f"Устройство {device_id} не найдено")
events.record("device.deleted", "device", d.id, f"Устройство {d.name} удалено", device_id=d.id,
data={"name": d.name, "host": d.host}, s=s)
s.delete(d)
def get_conn(device_id: str) -> Conn:
d = get_device(device_id)
return Conn(d.id, d.name, d.host, d.port, d.username,
security.decrypt(d.password_enc), d.verify_tls, d.use_tls)
def open_client(conn: Conn) -> RosClient:
return RosClient(conn.host, conn.port, conn.username, conn.password,
conn.verify_tls, get_settings().ros_timeout, conn.use_tls,
connect_timeout=get_settings().ros_connect_timeout)
def has_ros_update(st: dict) -> bool:
return version_newer(st.get("ros_latest"), st.get("ros_installed"))
def ros_state(st: dict) -> str:
"""Состояние колонки «Upgrade ROS»: unknown — нет данных проверки; check_error — последняя проверка
не удалась (старая latest-version могла сохраниться); update — есть обновление; downgrade — версия
канала старше установленной (long-term/testing после переключения канала); иначе current."""
if not (st.get("ros_latest") and st.get("ros_installed")):
return "unknown"
if st.get("ros_check_error"):
return "check_error"
if version_newer(st["ros_latest"], st["ros_installed"]):
return "update"
if version_newer(st["ros_installed"], st["ros_latest"]):
return "downgrade"
return "current"
def fw_state(st: dict) -> str:
"""Состояние колонки «Upgrade FW» (изменение 027): unknown — нет fw_current/fw_upgrade (CHR или не опрошено);
update — в установленную ROS встроена более новая прошивка; downgrade — записанная на плате прошивка новее
встроенной в ROS (например, после отката ROS, изменение 023 — плата не откатывается вместе с ROS); иначе
current. По образцу ros_state."""
if not (st.get("fw_current") and st.get("fw_upgrade")):
return "unknown"
if version_newer(st["fw_upgrade"], st["fw_current"]):
return "update"
if version_newer(st["fw_current"], st["fw_upgrade"]):
return "downgrade"
return "current"
def has_fw_update(st: dict) -> bool:
return fw_state(st) == "update"
def filter_devices(devs: list[Device], group: str = "", q: str = "", status: str = "",
updates: str = "", channel: str = "") -> list[Device]:
"""Общий фильтр списка устройств для UI и API (по кэшу статуса).
group: "" — все, "none" — без группы, иначе id группы. status: online/offline.
updates: ros / fw / any (есть обновление) / none (обновлений нет).
"""
q = q.strip().lower()
out = []
for d in devs:
st = d.status
if group == "none" and d.group_id is not None:
continue
if group not in ("", "none") and str(d.group_id) != group:
continue
if q and q not in " ".join((d.name, d.host, st.get("model") or "")).lower():
continue
if status == "online" and d.online is not True:
continue
if status == "offline" and d.online is True:
continue
ros_new, fw_new = has_ros_update(st), has_fw_update(st)
if (updates == "ros" and not ros_new) or (updates == "fw" and not fw_new):
continue
if (updates == "any" and not (ros_new or fw_new)) or (updates == "none" and (ros_new or fw_new)):
continue
if channel and st.get("channel") != channel:
continue
out.append(d)
return out