Added named/bind9 Application, Graphs, Check_MK-script #413

Merged
merged 2 commits into from Jan 17, 2015

Projects

None yet

2 participants

@f0o
Member
f0o commented Jan 16, 2015

#411
#412

Added named/bind9 Application, Graphs, Check_MK-script.

Configuration for Bind9:

options {
   ...
   statistics-file "/etc/bind/named.stats";
   zone-statistics yes;
   ...
};

Permission denied resolve: touch /etc/bind/named.stats && chown bind:bind /etc/bind/named.stats
(Substitute bind:bind with the user and group that runs your bind)

@laf
Member
laf commented Jan 16, 2015

Couple of things on this (I can't get it working as is).

In includes/polling/applications/bind.inc.php I had to change:

-if( !empty($agent_data['app']['bind']) && $app['app_id'] > 0 ) {
+if( !empty($agent_data['app']['bind'])) {

As $app['app_id'] isn't set and I can't actually see where this should be set. Not sure what you're thoughts are on this but out of the box it doesn't work for me :(

3 not defined variables in scrut output probably just need to be addressed and finally....

On the actual BIND graph page I would change:

bind - Queries

to

BIND - Queries

Just because BIND is supposed to be in caps.

@laf laf added the enhancement label Jan 16, 2015
@f0o
Member
f0o commented Jan 17, 2015

the app_id is set when you add the application under the device settings. I added the check to avoid having a file named 'app-bind-.rrd' being created and fed.

Scrut fixes and text rewrite incomming :)

Edit: worth to mention is that graphs require an App-ID as well so graphing in general wont work without.
We should think of a autodiscovery mechanism to automatically add App-IDs when it's missing.

@laf
Member
laf commented Jan 17, 2015

God I must have been blind last night. All working again this morning!!

One thing to add before the merge is to put the config options you need into the Agent.md doc for people to be able to set this up.

@f0o
Member
f0o commented Jan 17, 2015

Updated :)

@laf laf merged commit e7c6951 into librenms:master Jan 17, 2015

1 check passed

Scrutinizer 7 new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment