moira icon indicating copy to clipboard operation
moira copied to clipboard

Refactor event creation function in checker

Open borovskyav opened this issue 5 years ago • 1 comments

Сейчас код в этой функции ужасен:

  • Он плохо читается, его очень тяжело написан, его очень тяжело дописывать.
  • Из первого следует второе: в нем есть много несколько не очень приятных багов, например если состояние метрики меняется во время maintenace интервала то все следующие чеки триггера приводят к изменению времени последнего эвента этой метрики.
  • Функции создания эвента для метрики и триггера почти полностью дублируют свою функционнальность.

Всё усложняется тем, что у нас есть четко установленные правила на постановку maintenace и то, в каких случаях мы отправляем сообщения, но нет таких правил для изменения времени последнего эвента метрики. Нужно выработать эти правила и зафиксировать.

borovskyav avatar Apr 04 '19 19:04 borovskyav

Один из багов, который я пытался починить https://github.com/moira-alert/moira/pull/332

borovskyav avatar Apr 07 '19 12:04 borovskyav