Bug 26315 - Изменить реакцию в roorc
Summary: Изменить реакцию в roorc
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: monit (show other bugs)
Version: unstable
Hardware: all Linux
: P3 minor
Assignee: Michael Shigorin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-15 17:48 MSK by Gulay Boris
Modified: 2011-09-22 17:47 MSK (History)
2 users (show)

See Also:


Attachments
Изменённый файл с alert вместо unmonitor. (160 bytes, text/plain)
2011-09-21 19:26 MSK, Gulay Boris
no flags Details
Правльный файл настроект (failed -> changed). (160 bytes, text/plain)
2011-09-22 16:36 MSK, Gulay Boris
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gulay Boris 2011-09-15 17:48:51 MSK
Сейчас с шаблоне rootrc, который используется для мониторинга конфигов, на проверку контрольной суммы стоит действие unmonitor. Таким образом, при изменении конфига (не такая уж редкая и страшная операция) админ вынужден просить monit начать заново следить за файлом.
Мне кажется, что разумнее использовать действие alert, ибо изменение конфигов есть не такое уж и редкое действие. Было бы логично, если бы monit просто предупреждал об изменении письмом, а не бросал следить за изменённым файлом совсем.
Comment 1 Michael Shigorin 2011-09-15 21:12:16 MSK
Пожалуй, хотя хорошо бы перечитать документацию -- не зря же так сделали.
Приложите проверенный пример?
Comment 2 Gulay Boris 2011-09-21 19:25:21 MSK
Ну, если не понятно из текста, приложу. Что же ещё остаётся.
Хочу отметить, что все изменения, которые я внёс - это поменял unmonitor на alert.
Comment 3 Gulay Boris 2011-09-21 19:26:13 MSK
Created attachment 5117 [details]
Изменённый файл с alert вместо unmonitor.
Comment 4 Michael Shigorin 2011-09-21 19:52:24 MSK
Оно-то понятно, да сапог в бою надёжней ;-)  Спасибо, сейчас.
Comment 5 Repository Robot 2011-09-21 20:23:45 MSK
monit-5.3-alt2 -> sisyphus:

* Wed Sep 21 2011 Michael Shigorin <mike@altlinux> 5.3-alt2
- rootrc: checksum test failure now results in an alert event
  instead of unmonitor one (closes: #26315)
- EXAMPLES: added dovecot, fail2ban, mdadm, spawn-fcgi, unbound,
  also courtesy of Gulay Boris (closes: #25303)
Comment 6 Gulay Boris 2011-09-22 16:35:34 MSK
Ошибочка.
Помимо реации (unmonitor -> alert) нужно ещё изменить и тип мониторинга: failed -> changed.
Правильный файл прилагается.
Comment 7 Gulay Boris 2011-09-22 16:36:37 MSK
Created attachment 5119 [details]
Правльный файл настроект (failed -> changed).
Comment 8 Michael Shigorin 2011-09-22 17:01:28 MSK
Вот потому и предлагал вместо диффа на словах приложить проверенный файл. :)
Comment 9 Repository Robot 2011-09-22 17:47:17 MSK
monit-5.3-alt3 -> sisyphus:

* Thu Sep 22 2011 Michael Shigorin <mike@altlinux> 5.3-alt3
- fixed thinko in proposed rootrc (closes: #26315)