check_dig: stick with integer devision (e33ecc8)

Holger Weiß notifications at github.com
Thu Jan 30 21:52:25 CET 2014


Actually, I was inclined to hardcode `+tries=1` on the `dig(1)` command line.  While the `dig(1)` command happens to implement retries by default, most of our plugins don't, because users might want to *notice* if some fraction of queries fail, and because Nagios & friends support "soft states" if users *don't* want to be notified in such cases.  I'd be surprised if the original `check_dig` author consciously relied on these `dig(1)` semantics.

And yes, maybe the disagreement is "philosophical": I do not at all believe that more configuration switches are necessarily better.  Only those that might actually be useful are good :-)  But I'll agree that there are many cases where you could argue one way or the other.

Either way, if there's just one more comment on this single line of code, I'll revert to the floating point division, which I agree is totally fine as well ;-)

---
Reply to this email directly or view it on GitHub:
https://github.com/monitoring-plugins/monitoring-plugins/commit/e33ecc84c7ebdf4af3e8649a326e8a5acc9fe5b6#commitcomment-5230669
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.monitoring-plugins.org/archive/devel/attachments/20140130/801e943d/attachment.html>


More information about the Devel mailing list