Commit graph

15 commits

Author SHA1 Message Date
Ryan G. Hunter
67967d04ee monit startup fix 2016-12-08 11:34:11 -05:00
Brian Coca
4db5119893 corrected version_added for new monit timeout opt 2016-12-08 11:33:38 -05:00
Björn Andersson
d577fedc26 monit: Set a high timeout waiting for status changes
Instead of waiting for up to a certain number of retries we set a high
timeout and only re-check every five seconds. Certain services can
take a minute or more to start and we want to avoid waisting resources
by polling too often.
2016-12-08 11:33:38 -05:00
Björn Andersson
69e9bd3406 monit: Add version_added and type for new argument 2016-12-08 11:33:38 -05:00
Björn Andersson
757c9bffee monit: Wait for pending state changes for reloads
@mpeters reported that we're not checking that the named service is
actually there after a reload. And that sometimes monit doesn't actually
return anything at all after a reload.
2016-12-08 11:33:38 -05:00
Björn Andersson + SU Sheng Loong
469bf9ebd0 monit: Add retry for pending/initializing services
If there are already ongoing actions for a process managed by monit, the
module would exit unsuccessfully. It could also give off false positives
because it did not determine whether the service was started/stopped
when it was in a pending state. Which might be turning the service off,
but the action was to start it.

For example "Running - pending stop" would be regarded as the service
running and "state=enabled" would do nothing.

This will make Ansible wait for the state to finalize, or a timeout decided
by the new `max_retries` option, before it decides what to do.

This fixes issue #244.
2016-12-08 11:33:38 -05:00
Greg DeKoenigsberg
c65a612d1f Changes to author formatting, remove emails 2016-12-08 11:32:51 -05:00
Lorenzo Luconi Trombacchi
7817b69a1f status function was called twice 2016-12-08 11:32:47 -05:00
Lorenzo Luconi Trombacchi
83667961bb fix a problem with status detection after unmonitor command 2016-12-08 11:32:47 -05:00
Lorenzo Luconi Trombacchi
9ee857180b added lower function for statuses 2016-12-08 11:32:47 -05:00
Brian Coca
c4f52f62f7 fix doc parsing by correctly quoting author 2016-12-08 11:32:39 -05:00
Greg DeKoenigsberg
425989831f Add 2016-12-08 11:32:38 -05:00
Rob White
e6eb7d47eb Keep 'process' lowercase to protect against upstream changes 2016-12-08 11:32:24 -05:00
Rob White
ef8ca7c063 Changed status() to be case-sensitive of process 2016-12-08 11:32:24 -05:00
Michael DeHaan
5879c503bb File extensions! 2016-12-08 11:32:18 -05:00
Renamed from lib/ansible/modules/extras/monitoring/monit (Browse further)