phlipper / chef-monit

Chef cookbook for monit package
http://phlippers.net/chef-monit
MIT License
67 stars 69 forks source link

Use 'use_inline_resources' to ensure notifications propagate correctly #89

Open wlonkly opened 7 years ago

wlonkly commented 7 years ago

PR #69 updated the monit provider to use run_action on the provided resources to make external notifications propagate (i.e., so that a 'notifies' in a monit_monitrc's block would run). But that has the side effect of not firing the 'notifies' inside the monit_monitrc resource, and thus monit never reloads when a new monit_monitrc resource is created.

This uses use_inline_resources to evaluate the monit template in its own context, so that that context will fire notifications declared on the monit_monitrc resource, while still firing the notification on the template itself.

Tested against Chef 12.14.68, 12.21.1 (current) , 13.1.31 (current).