checkresults .ok file accumulation bugfix + added debug messages #295

Merged
merged 2 commits into from Dec 12, 2016

Conversation

Projects
None yet
2 participants
@caronc
Contributor

caronc commented Nov 17, 2016

The Debug messages are really just a bi-product of trying to figure out why I couldn't create custom passive-checks. They were being swallowed up but nothing in the debug logs (Even with full debugging verbosity set). Turns out it was a simple permission issue.

However a second bug was uncovered as a result of this permission issue causing the .ok files (in the checkresults directory) to accumulate and not be cleaned up.

The bugfix for this is included as well (a simple one-liner).

@caronc caronc changed the title from checkresults .ok file accumulation bugfix + added debug switches to checkresults .ok file accumulation bugfix + added debug messages Nov 17, 2016

@jfrickson jfrickson added the Bug label Nov 21, 2016

@jfrickson jfrickson self-assigned this Nov 21, 2016

@jfrickson jfrickson added this to the 4.3.0 milestone Nov 21, 2016

@jfrickson jfrickson changed the base branch from master to maint Dec 12, 2016

@jfrickson jfrickson merged commit ab8123b into NagiosEnterprises:maint Dec 12, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment