X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=config%2Fbuild-nagios;h=28513c96ec350e41546843be6a710a48ddd269f7;hb=703b7fae4acd1db3e71077a77127e0263b95d86c;hp=a940b877b0fc657501b80061f45d815b3dce6312;hpb=bd2cebc8abccd3df9143cdeee9e925040a75e1e8;p=mirror%2Fdsa-nagios.git

diff --git a/config/build-nagios b/config/build-nagios
index a940b87..28513c9 100755
--- a/config/build-nagios
+++ b/config/build-nagios
@@ -131,9 +131,7 @@ $nrpe = Nrpe.new()
 def print_block(fd, kind, hash, exclude_keys)
 	fd.puts "define #{kind} {"
 	hash.keys.sort.each{ |key|
-		next if key == '_extinfo'
-		next if key == '_hostgroups'
-		next if key == '_memberlist'
+		next if key[0,1] == '_'
 		next if exclude_keys.include? key
 		fd.puts "	#{key}		#{hash[key]}"
 	}
@@ -176,9 +174,13 @@ def addService(hosts, service, files, servers)
 		set_complain_if_set s, 'host_name', host, 'Service', s['service_description']
 		merge_contacts(servers[host], s)
 
-		print_block files['services'], 'service', s, %w(nrpe runfrom remotecheck
-		                                                depends
-		                                                hosts hostgroups excludehosts excludehostgroups)
+		exclude_keys = %w(nrpe runfrom remotecheck
+		                  depends
+		                  hosts hostgroups excludehosts excludehostgroups)
+		if servers[host]['no-servicegroups']
+			exclude_keys << 'servicegroups'
+		end
+		print_block files['services'], 'service', s, exclude_keys
 	}
 
 	if service['depends']
@@ -330,22 +332,24 @@ config['servers'].keys.sort.each{ |name|
 	set_if_unset        server, 'use'          , HOST_TEMPLATE_NAME
 	set_if_unset        server, 'check_command', HOST_ALIVE_CHECK    unless server['pingable'] == false
 
-	print_block files['hosts']      , 'host'       , server            , %w(hostgroups pingable)
+	print_block files['hosts']      , 'host'       , server            , %w(hostgroups pingable no-servicegroups)
 
 
 
 	# Handle hostextinfo
-	#config['hostgroups'][  server['_hostgroups'].first  ]['_extinfo'].each_pair{ |k, v|
-	# find the first hostgroup that has extinfo
-	extinfo = server['_hostgroups'].collect{ |hgname | config['hostgroups'][hgname]['_extinfo'] }.delete_if{ |ei| ei.size == 0 }.first
-	if extinfo then
-		extinfo.each_pair do |k, v|
-			# substitute hostname into the notes_url
-			v = sprintf(v,name) if k == 'notes_url'
-
-			set_if_unset server['_extinfo'], k ,v
+	# Compose extinfo from all the hostgroups for each server
+	config['hostgroups'].each_pair{ |hgname, hostgroup|
+		next unless server['_hostgroups'].include?(hgname)
+		hostgroup['_extinfo'].each_pair do |k, v|
+			if v.empty?
+				server['_extinfo'].delete(k)
+			else
+				# substitute hostname into the notes_url
+				v = sprintf(v,name) if k == 'notes_url'
+				server['_extinfo'][k] = v
+			end
 		end
-	end
+	}
 
 	set_complain_if_set server['_extinfo'], 'host_name'       , name, 'hostextinfo', name
 	set_if_unset        server['_extinfo'], 'vrml_image'      , server['_extinfo']['icon_image'] if server['_extinfo'].has_key?('icon_image')
@@ -380,6 +384,8 @@ config['services'].each{ |service|
 
 	split_away_extinfo service
 
+	# make sure the depends list is an array
+	service['depends'] = ensure_array( service['depends'] )
 
 	# Both 'name' and 'service_description' are valid for a service's name
 	# Internally we only use service_description as that's nagios' official term