Issue #3764 has been updated by James Turnbull. Status changed from In Topic Branch Pending Merge to Requires CLA to be signed Assignee set to Alan Barrett
Hi Alan I'm just going through patches and spotted this one had slipped through the cracks. Could I get you to sign a Contributor License Agreement (see top right menu link) and if possible submit your patch as a GitHub pull request as described https://github.com/puppetlabs/puppet/blob/master/CONTRIBUTING.md. No problems if you can't do the latter as i'll put it into a commit and send it up myself if needed. Thanks! And apologies this has taken so long to get some action. ---------------------------------------- Feature #3764: Add "fail" parameter to notify resource https://projects.puppetlabs.com/issues/3764 Author: Alan Barrett Status: Requires CLA to be signed Priority: Normal Assignee: Alan Barrett Category: functions Target version: 2.7.x Affected Puppet version: 0.25.4 Keywords: Branch: I often want a puppet manifest to result in an error on the client. This is useful for marking "to do" items, where users should be reminded of shortcomings in the manifest. I have added a "fail" parameter to the notify resource. Usage is as follows: <pre> notify { "Feature X is not yet implemented": fail => true, } </pre> I attach a patch to implement this feature, but I have not added test cases. It would be nice to have "onlyif" and "unless" parameters, but I have not implemented those. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
