Antwort: Re: Re: [Nagiosplug-devel] additional function for check_disk [Virus checked]

[Matthias.Eble at kaufland.de]

nagiosplug-devel-admin at lists.sourceforge.net schrieb am 01.06.2005 
16:24:00:

> On Wed, 1 Jun 2005 Matthias.Eble at kaufland.de wrote:
> 
> > nagiosplug-devel-admin at lists.sourceforge.net schrieb am 17.05.2005
> > 19:52:09:
> >
> >> Matthias.Eble at kaufland.de wrote:
> >>> Hi list,
> >>>
> >>>  at the moment, check_disk returns a state != STATE_OK if any of the
> >>> monitored filesystems has less than -w or -c freespace.
> >>>
> >>> In our case, this doesn't meet our needs, since our Oracle databases
> > use
> >>> more than just one filesystem to store their data.
> >>> So if there are n data-volumes available, it doesn't matter if one 
or
> > n-1
> >>> volumes get filled 100% as long as one volume remains beyond
> >>> the warning level.
> >>>
> >>> I've got three approaches for solve this problem.
> >>>
> >>> First:
> >>> Introduce an option like -g (--group) followed by a
> > comma_seperated_list
> >>> of partitions/lvols/mountpoints which are to group.
> >>>
> >>
> >> Probably the cleanest solution, or possibly to add check_disk_group 
as
> >> its own command.
> >>
> >
> > So after analyzing the code of check_disk it looks like it would be 
the
> > best way
> > to add two values to the mount_entry struct in mountlist.h:
> > State (STATE_OK,..)
> > gid   (group id)
> >
> >
> > I couldn't find any file (except check_disk.c) which uses the 
mount_entry
> > struct,
> > so I like to know if a change like this matches nagiosplug's coding 
rules.
> > I'm in doubt because it's located in the lib directory.
> >
> > Please be lenient toward me since this is my first contribution.. :)
> >
> >
> 
> a patch is always easier to accept as a contribution.
> 
well, maybe I chose the wrong word...
this will be my first contributed patch..