Commit Graph

271 Commits (5bedee21870bab2ca785c5e25d44cc40f37e6667)

Author SHA1 Message Date
Michael Scherer 2f3dc1352f Replace choices=BOOLEANS by type='bool', fix #1326 9 years ago
Brian Coca a6ceec998d corrected version_added for new monit timeout opt 9 years ago
Brian Coca 91df684902 Merge pull request #806 from gaqzi/monit/retry-option
monit: Add retry for pending/initializing services
9 years ago
Bruce Pennypacker 5abd7deab0 Remove extraneous space from end of command string in nagios_cmd 9 years ago
Michael Weber fba8c9f8a7 Fix error 'fail_json() takes exactly 1 argument'
Fixes bug #1257
9 years ago
Björn Andersson 72155d40a3 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.
9 years ago
Björn Andersson 262f2e9048 monit: Add version_added and type for new argument 9 years ago
Björn Andersson 5835d06a4e 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.
9 years ago
Björn Andersson + SU Sheng Loong f8fe5a2fcd 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.
9 years ago
Brian Coca 875a055103 corrected choices which was meant to be type 9 years ago
Ritesh Khadgaray 5e103d604a allows user to not update zabbix host config if host is present. 9 years ago
Kerim Satirli 2ac53bf559 fixes a typo in Datadog Monitor docs 9 years ago
Konstantin Manna 97b7a7a7f8 bugfix: use correct function close calls 9 years ago
Brian Coca 6cd7399a71 minor docfix 9 years ago
Brian Coca 076ddc5266 Merge pull request #1002 from knakayama/bugfix/fix-argument-type-for-zabbix_screen
Fix argument type for zabbix_screen
9 years ago
Brian Coca c7c32ef86f minor doc fixes nagios 9 years ago
Brian Coca 5017c4483a Merge pull request #798 from vaupelt/patch-1
strange results with services=all

agreed, the doc issues are unrelated to this PR, will fix those independently
9 years ago
knakayama 45f4568753 Fix argument type for zabbix_screen 9 years ago
Anders Ingemann e0f513a903 Fix state & metric description
Also: add state=absent to last example
9 years ago
Anders Ingemann 9342c16e78 Add github ID to author field in sensu_check
As proposed by @gregdek in #983
9 years ago
Darren Worrall b0926125c2 Fix json loading in sensu_check
Fixes `AttributeError: 'str' object has no attribute 'read'` when the
check path exists
9 years ago
Darren Worrall 4ead053031 Initialise `stream` variable
Fixes `UnboundLocalError: local variable 'stream' referenced before
assignment` when the check path doesnt exist
9 years ago
Tim Bielawa c9785a6948 Fix capitalization in nagios 'services' parameter comment 9 years ago
Amanpreet Singh a0af060c25 Make pagerduty_alert module more inline with ansible modules
- use state parameter instead of event_type
- add support for check mode
9 years ago
Amanpreet Singh 0847bfecd6 Add new module: pagerduty_alert
- trigger, acknowledge or resolve pagerduty incidents
9 years ago
vaupelt 150d3ce0ee strange results with services=all
I issued a command with action=disable_alerts host=webserver services=all set and get this results:

    "nagios_commands": [
        "[1438593631] DISABLE_SVC_NOTIFICATIONS;webserver;a", 
        "[1438593631] DISABLE_SVC_NOTIFICATIONS;webserver;l", 
        "[1438593631] DISABLE_SVC_NOTIFICATIONS;webserver;l"
    ]

This is not a big deal because i have just overlooked the action=silence command. Nevertheless a more predictable result would be a nice thing to have.
9 years ago
Brian Coca 27bf193483 added missing license headers
fixes #508
9 years ago
Toshio Kuratomi aa1b4caa8a Merge pull request #734 from ansible/fetch_url-uptimerobot
Port uptimerobot to fetch_url
9 years ago
Toshio Kuratomi 4a43f52437 Use fetch_url's basic auth instead of code specific to this module 9 years ago
Toshio Kuratomi 8aa490c638 Port uptimerobot to fetch_url 9 years ago
Toshio Kuratomi 626977f90e Port librarto_annotation from urllib2 to fetch_url 9 years ago
Toshio Kuratomi 79173ac18d Clean up unneeded urllib2 imports 9 years ago
Greg DeKoenigsberg d640fd95e9 Merge pull request #605 from bpennypacker/pagerduty_updates
monitoring/pagerduty module improvements, including specifying multiple services, fixing task output, adding 'delete'' action
9 years ago
Eike Frost 75e1e9fcda add zabbix proxy support to zabbix_host 9 years ago
Brian Coca 3f9e2fb4e6 Merge pull request #629 from bcoca/sensu_check_2.4
made sensu_check 2.4 friendly
9 years ago
Brian Coca 286bc3d9dc forgot finally 2.4 syntax 9 years ago
Brian Coca eeb9d34812 also fixed exceptions 9 years ago
Toshio Kuratomi dd6e8f354a Modify a few more modules to not conditionalize urllib(2) import. 9 years ago
Brian Coca 1659af1541 made sensu_check 2.4 friendly 9 years ago
Toshio Kuratomi 4b29146c4d be explicit about urllib import and remove conditional urllib(2) import
urllib and urllib2 have been in the python stdlib since at least
python-2.3.  There's no reason to conditionalize it.

Fixes https://github.com/ansible/ansible/issues/11322
9 years ago
Brian Coca 8f31d84300 Merge pull request #203 from andsens/sensu-check
Add sensu_check module
9 years ago
Anders Ingemann 3f3a73da37 Add sensu_check module 9 years ago
Greg DeKoenigsberg 004dedba8a Changes to author formatting, remove emails 9 years ago
Brian Coca 0d7332d550 minor docfix 9 years ago
Greg DeKoenigsberg 98abb6d2c9 Adding author's github id 9 years ago
Jhonny Everson 4b625bab34 Fixes the bug where it was using only the keys to determine whether a change was made, i.e. values changes for existing keys was reported incorrectly. 9 years ago
Jhonny Everson d722d6de97 Adds handler for error responses 9 years ago
Sebastian Kornehl fdaa4da447 docs: removed default when required is true 9 years ago
Sebastian Kornehl 1291f9a25a Added datadog_monitor module 9 years ago
Brian Coca 6050cc8e5d push list nature of tags into spec to allow both for comma delimited strings and actual lists 9 years ago