Conversation
nanodirijabl
left a comment
nanodirijabl
left a comment
There was a problem hiding this comment.
Не покидает ощущение что где-то не в ту сторону свернули и для потребителей доминанты потребуются дополнительные доработки помимо замены ещё и клиента.
| {elvis_project, no_deps_master_rebar, disable}, | ||
| {elvis_project, no_branch_deps, disable} |
There was a problem hiding this comment.
Кажется что это временные изменения в правилах, которые следует вернуть после бампа зависимости.
| }}, | ||
| {elvis_style, no_throw, disable}, | ||
| {elvis_style, export_used_types, disable} |
There was a problem hiding this comment.
Странно что эти правила надо отключать. erlang:throw было в проекте и до этого.
| health_check() -> | ||
| try | ||
| _ = dmt_client_cache:get_last_version(), | ||
| % TODO Come up with healthcheck |
There was a problem hiding this comment.
Это критически важный момент. Некоторые сервисы дёргают этот чек для прохождения рединес-пробы. От таких сервисов ожидается что они должны быть готовы обслуживать запросы сразу при готовности и соответственно та или иная доступность объектов доминанты для него является критическим признаком здоровья.
There was a problem hiding this comment.
Спасибо, что напомнил. Допилю.
3 participants
No description provided.