Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

алерты в Telegram падают вне зависимости от настроек в "watch time" #89

Closed
vraskoshnyy opened this issue Jun 9, 2018 · 16 comments

Comments

@vraskoshnyy
Copy link

Заметили странное поведение - алерты от Мойра приходят во интервал времени, который не входит в "watch time".
Т.е. например, в watch time указано "At specific interval: 12:00 - 23:59"
а уведомление в телегу прилетело в 10:09.
Проверьте, пожалуйста.
Спасибо.

@borovskyav
Copy link
Contributor

Привет! Мы постараемся на днях посмотреть, воспроизвести у себя, спасибо =)

@vraskoshnyy
Copy link
Author

Привет. Получилось?
У меня стабильно воспроизводится(

@vraskoshnyy
Copy link
Author

vraskoshnyy commented Aug 9, 2018

И ещё хотел добавить. Триггер может быть со статусом error в watch time, которое нам не интересно (за его границами), но при этом на общем dashboard он будет показан с переключателем "Only problems". Кажется, что это не совсем логично. Т.е. если мне не нужно знать состояние триггера в период, например, с 00 до 07, то зачем его показывать как "проблему"? Может сделать системную настройку, чтобы можно было скрывать такие триггеры по времени и на общем dashboard?

@borovskyav
Copy link
Contributor

Привет! Так увлеклись релизом 2.3 что совсем забыли. На следующей неделе зарелизим 2.3 и постараемся заняться проверкой бага одной из первых задач. Сории за ожидание(

@borovskyav
Copy link
Contributor

borovskyav commented Aug 9, 2018

По поводу watch time - спасибо за реквест, мы сейчас активно пречесываем логику с мейнтенансом метрик в триггере, в релизе 2.3 немного переработали механику работы мейнтенанса. В 2.4 продолжим причесывать, накопилось уже несколько разных реквестов, например https://github.com/moira-alert/web2.0/issues/153. Добавили твой реквест в список. Там кстати один из реквестов - фильтровать по триггерам, у которых нет мейнтенанса, так что думаю эти 2 задачи в целом очень похожи, но мы еще подумаем про красивое решение)

@borovskyav
Copy link
Contributor

@vraskoshnyy Кстати, будет круто если подскажешь какая таймзона проставлена у тебя этой настройке watchTime. Это можно посмотреть так: GET moira-host/api/trigger/{triggerID} (или в хроме в режиме разработчика при загрузке этого триггера можно посмотреть результат запроса), интересует поле sched.tzOffset. И еще подскажи, в каком часовом поясе ты живешь.

@vraskoshnyy
Copy link
Author

Привет, я живу в MSK.
"tzOffset":-300"

@borovskyav
Copy link
Contributor

Мы нашли баг. У тебя этот offset должен быть -180. Баг в проставлении при сохранении триггера, сейчас поправим. В релиз 2.3 попадет

@vraskoshnyy
Copy link
Author

Круто, спасибо)

@borovskyav
Copy link
Contributor

borovskyav commented Aug 13, 2018

moira-alert/web2.0#172 Как обновитесь до 2.3 нужно будет пересохранить триггер

@vraskoshnyy
Copy link
Author

Понял, спасибо.

@sashasushko
Copy link
Contributor

Fixed on front.

@titusjaka
Copy link
Contributor

moira-alert/web2.0#173

@sashasushko
Copy link
Contributor

нужно будет пересохранить триггер

This may not help. I check today trigger edit logic.

Because, I think, we don't change timezone, when user edit trigger. It's unobvious behavior. So... we need to discuss it before change this logic.

@sashasushko
Copy link
Contributor

Unfortunately, it's true. Timezone set once when trigger create.

@vraskoshnyy you need to recreate trigger.

We will think how fix this behavior in next release.

@vraskoshnyy
Copy link
Author

@sashasushko ок, пересоздам

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants