sensu 0.20.5-java → 0.20.6-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 923c6f4227eaf088c354a1537d7caf9b6d52bd43
-  data.tar.gz: bc2c5489cbb22cda8ab7ac03f7954c424448fb22
+  metadata.gz: c0a54f2050a55757840257922d75b5fc25b2ecb2
+  data.tar.gz: f74d4efbc2ba470a991b09fc2fe3e1d026c28364
 SHA512:
-  metadata.gz: 57a1da9ffca503307405c449972c93fe7af542e4f2bb754f4504a6ae36cce4409cbd64797a0ac9d61a42bb6f4ef2756c6fc90a5327dc9a349871500d60837a06
-  data.tar.gz: 83415bcd10f4f138817e0f31e6fd9f096e9d0766094bd632adfc09ea32c5501914becf8d1a2cfc2117d7f142fbd89dbe1a489105f149df31e4ad29c8f5042acd
+  metadata.gz: 6432bd38ff20bbd9879b8bdfbf54a525d24620adff8f34625a2b182a0da0db8842afe65930447999379ecc83ff12baeb143b8410d283a2a634d7a24b6b2dedf6
+  data.tar.gz: b2e98918ef1b2e8ae76e7bb84335a5b7e33adbbf8380f080ff1e032a94dad7e17be6316098137faeee0341153b3b845b8c363e3fbf25543ec01b9cc138a4fe27

data/CHANGELOG.md

@@ -1,3 +1,21 @@
+## 0.20.6 - 2015-09-22
+
+### Other
+
+Removed the use of `EM::Iterator` from event filtering, replacing it with
+`Proc` and `EM::next_tick`. `EM::Iterator` creates anonymous classes that
+cannot be garbage collected on JRuby.
+
+Removed sensu-em as a dependency, now using upstream EventMachine 1.0.8.
+
+The Sensu API will remove a client immediately if there are no current
+events for it. The API will continue to monitor the current event count
+for the client to be deleted, deleting the client when there are no longer
+current events or after a timeout of 5 seconds.
+
+The Sensu API will no longer crash while fetching check result data for a
+client that is being deleted.
+
 ## 0.20.5 - 2015-09-09
 
 ### Other

data/lib/sensu/api/process.rb

@@ -337,9 +337,7 @@ module Sensu
                 unless client_json.nil?
                   response << MultiJson.load(client_json)
                 else
-                  settings.logger.error("client registry missing data for client", {
-                    :client_name => client_name
-                  })
+                  settings.logger.error("client data missing from registry", :client_name => client_name)
                   settings.redis.srem("clients", client_name)
                 end
                 if index == clients.size - 1
@@ -375,17 +373,19 @@ module Sensu
                   status.to_i
                 end
                 settings.redis.get("result:#{result_key}") do |result_json|
-                  result = MultiJson.load(result_json)
-                  last_execution = result[:executed]
-                  unless history.empty? || last_execution.nil?
-                    item = {
-                      :check => check_name,
-                      :history => history,
-                      :last_execution => last_execution.to_i,
-                      :last_status => history.last,
-                      :last_result => result
-                    }
-                    response << item
+                  unless result_json.nil?
+                    result = MultiJson.load(result_json)
+                    last_execution = result[:executed]
+                    unless history.empty? || last_execution.nil?
+                      item = {
+                        :check => check_name,
+                        :history => history,
+                        :last_execution => last_execution.to_i,
+                        :last_status => history.last,
+                        :last_result => result
+                      }
+                      response << item
+                    end
                   end
                   if index == checks.size - 1
                     body MultiJson.dump(response)
@@ -406,22 +406,31 @@ module Sensu
               events.each do |check_name, event_json|
                 resolve_event(event_json)
               end
-              EM::Timer.new(5) do
-                client = MultiJson.load(client_json)
-                settings.logger.info("deleting client", :client => client)
-                settings.redis.srem("clients", client_name) do
-                  settings.redis.del("client:#{client_name}")
-                  settings.redis.del("events:#{client_name}")
-                  settings.redis.smembers("result:#{client_name}") do |checks|
-                    checks.each do |check_name|
-                      result_key = "#{client_name}:#{check_name}"
-                      settings.redis.del("result:#{result_key}")
-                      settings.redis.del("history:#{result_key}")
+              delete_client = Proc.new do |attempts|
+                attempts += 1
+                settings.redis.hgetall("events:#{client_name}") do |events|
+                  if events.empty? || attempts == 5
+                    settings.logger.info("deleting client from registry", :client_name => client_name)
+                    settings.redis.srem("clients", client_name) do
+                      settings.redis.del("client:#{client_name}")
+                      settings.redis.del("events:#{client_name}")
+                      settings.redis.smembers("result:#{client_name}") do |checks|
+                        checks.each do |check_name|
+                          result_key = "#{client_name}:#{check_name}"
+                          settings.redis.del("result:#{result_key}")
+                          settings.redis.del("history:#{result_key}")
+                        end
+                        settings.redis.del("result:#{client_name}")
+                      end
+                    end
+                  else
+                    EM::Timer.new(1) do
+                      delete_client.call(attempts)
                     end
-                    settings.redis.del("result:#{client_name}")
                   end
                 end
               end
+              delete_client.call(0)
               issued!
             end
           else
@@ -744,8 +753,10 @@ module Sensu
                   checks.each_with_index do |check_name, check_index|
                     result_key = "result:#{client_name}:#{check_name}"
                     settings.redis.get(result_key) do |result_json|
-                      check = MultiJson.load(result_json)
-                      response << {:client => client_name, :check => check}
+                      unless result_json.nil?
+                        check = MultiJson.load(result_json)
+                        response << {:client => client_name, :check => check}
+                      end
                       if client_index == clients.size - 1 && check_index == checks.size - 1
                         body MultiJson.dump(response)
                       end
@@ -769,8 +780,10 @@ module Sensu
             checks.each_with_index do |check_name, check_index|
               result_key = "result:#{client_name}:#{check_name}"
               settings.redis.get(result_key) do |result_json|
-                check = MultiJson.load(result_json)
-                response << {:client => client_name, :check => check}
+                unless result_json.nil?
+                  check = MultiJson.load(result_json)
+                  response << {:client => client_name, :check => check}
+                end
                 if check_index == checks.size - 1
                   body MultiJson.dump(response)
                 end

data/lib/sensu/constants.rb

@@ -1,7 +1,7 @@
 module Sensu
   unless defined?(Sensu::VERSION)
     # Sensu release version.
-    VERSION = "0.20.5"
+    VERSION = "0.20.6"
 
     # Sensu check severities.
     SEVERITIES = %w[ok warning critical unknown]

data/lib/sensu/daemon.rb

@@ -1,14 +1,14 @@
 require "rubygems"
 
 gem "multi_json", "1.11.2"
+gem "eventmachine", "1.0.8"
 
-gem "sensu-em", "2.5.2"
-gem "sensu-logger", "1.0.0"
+gem "sensu-logger", "1.1.0"
 gem "sensu-settings", "3.1.0"
-gem "sensu-extension", "1.1.2"
-gem "sensu-extensions", "1.2.0"
-gem "sensu-transport", "3.2.1"
-gem "sensu-spawn", "1.4.0"
+gem "sensu-extension", "1.2.0"
+gem "sensu-extensions", "1.3.0"
+gem "sensu-transport", "3.3.0"
+gem "sensu-spawn", "1.5.0"
 
 require "time"
 require "uri"

data/lib/sensu/server/filter.rb

@@ -259,16 +259,17 @@ module Sensu
       def event_filtered?(handler, event, &callback)
         if handler.has_key?(:filters) || handler.has_key?(:filter)
           filter_list = Array(handler[:filters] || handler[:filter])
-          filter_results = EM::Iterator.new(filter_list)
-          run_filters = Proc.new do |filter_name, iterator|
-            event_filter(filter_name, event) do |filtered|
-              iterator.return(filtered)
+          filter = Proc.new do |filter_list|
+            filter_name = filter_list.shift
+            if filter_name.nil?
+              callback.call(false)
+            else
+              event_filter(filter_name, event) do |filtered|
+                filtered ? callback.call(true) : EM.next_tick { filter.call(filter_list) }
+              end
             end
           end
-          filtered = Proc.new do |results|
-            callback.call(results.any?)
-          end
-          filter_results.map(run_filters, filtered)
+          EM.next_tick { filter.call(filter_list) }
         else
           callback.call(false)
         end

data/sensu.gemspec

@@ -16,14 +16,13 @@ Gem::Specification.new do |s|
   s.add_dependency "json" if RUBY_VERSION < "1.9"
   s.add_dependency "multi_json", "1.11.2"
   s.add_dependency "uuidtools", "2.1.5"
-  s.add_dependency "eventmachine", "1.0.3"
-  s.add_dependency "sensu-em", "2.5.2"
-  s.add_dependency "sensu-logger", "1.0.0"
+  s.add_dependency "eventmachine", "1.0.8"
+  s.add_dependency "sensu-logger", "1.1.0"
   s.add_dependency "sensu-settings", "3.1.0"
-  s.add_dependency "sensu-extension", "1.1.2"
-  s.add_dependency "sensu-extensions", "1.2.0"
-  s.add_dependency "sensu-transport", "3.2.1"
-  s.add_dependency "sensu-spawn", "1.4.0"
+  s.add_dependency "sensu-extension", "1.2.0"
+  s.add_dependency "sensu-extensions", "1.3.0"
+  s.add_dependency "sensu-transport", "3.3.0"
+  s.add_dependency "sensu-spawn", "1.5.0"
   s.add_dependency "em-redis-unified", "1.0.0"
   s.add_dependency "sinatra", "1.4.6"
   s.add_dependency "async_sinatra", "1.2.0"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sensu
 version: !ruby/object:Gem::Version
-  version: 0.20.5
+  version: 0.20.6
 platform: java
 authors:
 - Sean Porter
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-09-09 00:00:00.000000000 Z
+date: 2015-09-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -45,26 +45,12 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.3
+        version: 1.0.8
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.3
-  prerelease: false
-  type: :runtime
-- !ruby/object:Gem::Dependency
-  name: sensu-em
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 2.5.2
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 2.5.2
+        version: 1.0.8
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency
@@ -73,12 +59,12 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.0
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.0
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency
@@ -101,12 +87,12 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.1.2
+        version: 1.2.0
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.1.2
+        version: 1.2.0
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency
@@ -115,12 +101,12 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: 1.3.0
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: 1.3.0
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency
@@ -129,12 +115,12 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.2.1
+        version: 3.3.0
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.2.1
+        version: 3.3.0
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency
@@ -143,12 +129,12 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 1.5.0
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 1.5.0
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency