Nagios Support Forum

Ah, no. The really sensible thing to do would
be to wait only until all the blocking checks
are done (either just one of "too many", or
all other checks in the parallelization case).
Sleeping for a full second regardless of when
the blocking checks complete can waste time
between when the next plugin could run and
when it actually does. And with enough checks
introducing these extra arbitrary delays, the
overall latency for the full set of checks can
easily creep up.



Thats a good point, so the better thing might be to move that check to the =
end of the event_list and move the even_list right along.
I have some experimental code I'm working with.
If you put it in above the place where it goes to sleep, it should unblock =
the queue and let the other checks run.

//Move this event to the end of the queue so it doesn't block
if(event_list_low->event_type=3D=3DEVENT_SERVICE_CHECK)
{
timed_event *this_event =3D temp_event =3D event_list_l=
ow;
event_list_low =3D event_list_low->next;

while(temp_event->next)
{
temp_event =3D temp_event->next;
}

temp_event->next =3D this_event;
this_event->next =3D NULL;

}

In my sources I placed it at about line 1112 or 1113 in events.c but YMMV s=
ince my sources have some extra patches and stuff in them that may have shi=
fted the line numbers.

Let me know what you think.

Sincerely,
Steven Morrey


NOTICE: This email message is for the sole use of the intended recipient(s=
) and may contain confidential and privileged information. Any unauthorized=
review, use, disclosure or distribution is prohibited. If you are not the =
intended recipient, please contact the sender by reply email and destroy al=
l copies of the original message.







This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]