lib/flapjack/gateways/pagerduty.rb in flapjack-1.6.0rc1 vs lib/flapjack/gateways/pagerduty.rb in flapjack-1.6.0rc2

@@ -172,14 +172,19 @@
         end
         [status, response]
       end
 
       def find_pagerduty_acknowledgements
-        @logger.debug("looking for acks in pagerduty for unack'd problems")
-
         unacknowledged_failing_checks = Flapjack::Data::EntityCheck.unacknowledged_failing(:redis => @redis)
+        if unacknowledged_failing_checks.empty?
+          @logger.debug("skipping looking for acks in pagerduty as there are no unacknowledged failing checks")
+          return nil
+        else
+          @logger.debug("looking for acks in pagerduty for unack'd problems")
+        end
 
-        @logger.debug "found unacknowledged failing checks as follows: " + unacknowledged_failing_checks.join(', ')
+        @logger.debug "found #{unacknowledged_failing_checks.length} unacknowledged failing checks as follows: " +
+          unacknowledged_failing_checks.map {|c| "#{c.entity_name}:#{c.check}"}.join(', ')
 
         unacknowledged_failing_checks.each do |entity_check|
 
           # If more than one contact for this entity_check has pagerduty
           # credentials then there'll be one hash in the array for each set of