nikita-p
commented
1 month ago - я бы наверное внёс логику в
HealthControl, т.к. там уже у нас происходит вся проверка статусов- плюсы: не надо лишний раз гонять данные со статусами от девбэка к чеку, вся проверка статусов в одном месте;
- тогда эта проверка была бы в блоке, который работает всегда (не только при автопилоте), что на самом деле хорошо, потому что данные, как мне кажется, никогда нельзя позволять набирать на статусах RampUp/Down (поскольку в эти интервалы времени система не стабильна), а если кто-то сильно хочет, то он сможет обойти это ограничение на стороне МЧС (насколько я помню, там можно вырубить проверку части программ)
- минусы: нельзя задавать разное время для разных проверок статусов (хотя я не думаю, что это критично), придётся чуть усложнить
HealthControlлогику, поскольку кроме действий {аварийный Down и NACK} или {всё ок, продолжаем наблюдать и ACK} будет {продолжаем наблюдение и NACK}
- ещё всё-таки продолжу пушить предложение слать NACK (в
HealthControl) при наблюдаемом уровне мультипликатора напряжения ниже X (где X задаётся в конфиге), поскольку после аварийного срабатывания Down мы начинаем слать ACK, т.к. статусы восстанавливаются, а между тем, напряжения нет - про буфер: в принципе можно на стороне чека установить общение с девбэком только через прокси-функцию, у которой установлен временной кэш; в целом есть над чем подумать, но пока что, как мне кажется, это не то, что нас сдерживает
AliensWithdrawPeople
tl;dr: По требованию Попова изменена логика отпрапвления ACK на МЧС.
Логика: Если на каналах происходит изменение напряжения (например, интерлок снялся и напряжение начало восстановливаться), то пото данных, то есть МЧС, должен быть заблокирован. Разблокировка происходит после выхода на рабочий режим.
Как сделал: Long story short, костыльно. Дописал ещё один скрипт в автопилот, который считывает статусы с devback, проверяет соответсвующие биты и лочит МЧС, если кто-то из них поднят. Скрипт подефолту запускается каждую секунду.
Работает?: Не знаю, не успел потестить на железе. А без железа нужно как-то эффективно мокнуть мчс, чтобы видеть результат.
@nikita-p Ты лучше разбираешься в этой части кода.
One more thing: Мне не нравится, что у нас сейчас каждая собака в SysCheck по любому чиху лазит в devback. Пока всё на одной машине, это не страшно. Но как-то напряжно вяглядит. Учитывая, что лазят части одного микросервиса, можно представить какую-то единую систему обращения к devback с буффером для k последних запросов. Как думаешь?