[Nagiosplug-devel] [PATCH/PULL] New plugin: check_dbi

[Sebastian Harl]

Hi,

On Wed, Jun 06, 2012 at 12:38:42PM +0200, Holger Weiß wrote:
> * Sebastian Harl <tokkee at debian.org> [2012-06-06 11:55]:
> > On Fri, Apr 08, 2011 at 11:45:11AM +0200, Sebastian Harl wrote:
> > > The latest version of the plugin can be found at
> > > <http://oss.teamix.org/projects/nagiosplug/repository/entry/plugins/check_dbi.c?rev=sh%2Fcheck_dbi>.
> > > 
> > > I'm looking forward to comments, suggestions and (of course) patches :-)
> > 
> > As there are rumours about a new release, I'd like to point out this new
> > plugin again. Any feedback about whether this is interesting for the
> > nagiosplugins project and/or needs some more work would be appreciated.
> > If you prefer a Github pull request instead, I'd be happy to do that as
> > well.
> 
> While we currently don't have the resources to include many new plugins,
> I think this one _is_ interesting.  At first glance, the code looks fine
> to me.  If you (or someone else) provide some test cases for this
> plugin, and if nobody objects, I'd consider it for inclusion.

I've added a few test-cases and opened a pull request on Github. Imho,
that should be used for further comments, objections, etc. ;-)

> This won't make it into the upcoming release, but we could probably
> import it right afterwards.

Great! :-)

Cheers,
Sebastian

-- 
Sebastian "tokkee" Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/

Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety.         -- Benjamin Franklin

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://nagios-plugins.org/archive/devel/attachments/20120608/92f35130/attachment.sig>