Merge pull request sous-chefs#272 from mdxp/duplicate-roles-env

prevent duplicates hostgroups in case the same roles/environment exist
gabrielcrowdtilt · May 3, 2014 · 571f0b2 · 571f0b2
2 parents 50cde3e + a1709c1
commit 571f0b2
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/recipes/default.rb b/recipes/default.rb
@@ -136,7 +136,7 @@
 # if using multi environment monitoring add all environments to the array of hostgroups
 if node['nagios']['multi_environment_monitoring']
   search(:environment, '*:*') do |e|
-    hostgroups << e.name
+    hostgroups << e.name unless hostgroups.include?(e.name)
     nodes.select { |n| n.chef_environment == e.name }.each do |n|
       service_hosts[e.name] = n[node['nagios']['host_name_attribute']]
     end