search

naemon / naemon-core

Networks, Applications and Event Monitor
http://www.naemon.io/
GNU General Public License v2.0
154 stars 63 forks source link

Allow exclude (!) to work for regexp matches as well #13

Closed liedekef closed 10 years ago

liedekef commented 10 years ago

Since I'll be trying naemon as well, I'm hoping to include the following feature as well: for regex, the exclude ('!') doesn't work, so you can't say something like e.g. for hosts: .,!bt. (include every host, but exclude those starting with bt) The original patch (never made it into nagios I think) can still be found here: http://permalink.gmane.org/gmane.network.nagios.devel/5338 I updated this patch and sent it to icinga and it got included: https://dev.icinga.org/issues/3720 So, if you're interested, I can create a patch for naemon as well (or you guys can do it)?

dwittenberg2008 commented 10 years ago

Are you sure these weren’t working in Nagios 4? I was pretty sure I used to have a client I did a lot of excludes so I could have a big hostgroup, like all web servers, but then excluded a couple odd ball ones.

Dan

On Dec 28, 2013, at 5:42 PM, Franky Van Liedekerke notifications@github.com wrote:

Since I'll be trying naemo as well, I'm hoping to include the following feature as well: for regex, the exclude ('!') doesn't work, so you can't say something like e.g. for hosts: .,!bt. (include every host, but exclude those starting with bt) The original patch (never made it into nagios I think) can still be found here: http://permalink.gmane.org/gmane.network.nagios.devel/5338 I updated this patch and sent it to icinga and it got included: https://dev.icinga.org/issues/3720 So, if you're interested, I can create a patch for naemo as well (or you guys can do it)?

— Reply to this email directly or view it on GitHub.

liedekef commented 10 years ago

To be fair: I'm not sure about nagios 4, since I switched to icinga some time ago and running the icinga 1.x branch for now. Icinga2 seems to go to a xml-style of config files which imho is not easier to create, so I'm looking at naemon now :-) However I'm 100% sure it doesn't work in nagios 3.

ageric commented 10 years ago

On 2014-01-09 11:20, Franky Van Liedekerke wrote:

To be fair: I'm not sure about nagios 4, since I switched to icinga some time ago and running the icinga 1.x branch for now. Icinga2 seems to go to a xml-style of config files which imho is not easier to create, so I'm looking at naemon now :-) However I'm 100% sure it doesn't work in nagios 3.

It doesn't work in Nagios 4 or Naemon either, and patches would be welcome. I was looking into this exact thing yesterday.

/Andreas

liedekef commented 10 years ago

I created a pull request for this, patch to be reviewed of course.