sensu-plugins-kafka 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 436d7165e1ea6b4ac7efe7ee7c4456619cb0d7bd
-  data.tar.gz: 1b33367a5fb5e418031ab76b3f1f571c660d4e69
+  metadata.gz: 238d5d13127c16a53fd30c5a4dae4a05471547cf
+  data.tar.gz: 0d81d88567c829276fb5a9a11db4c9eae162f196
 SHA512:
-  metadata.gz: e0386a39236c73b7b192bd287e1cd00c7a02da9ef8af2e79f26315d49a3dc005e87ddc77f66e19bbc47fb0f96b519963001fa574e496758ef3df92a35ab8825c
-  data.tar.gz: b500208baf3951334ef8a055feb277cb27aa261f7f1ea0b21f4618ba86ba5ad0287abad5b462f00ad61c963c22defb08d7278ab15e2d28c78b2519b21c6e5ea6
+  metadata.gz: 41831c5fc35254acbf40b9cee04c8bf0735a9be6062f2a94f94b9552d8f85cd6fcfaa5356147ac369d7452965d7c1cba3c9a9dc2ea9478ee083a837f16cccc9c
+  data.tar.gz: 2eb96ee56a1888894765e8d28614bd185919a1542abd5ae028092506afc86b018cca1909041d3ac28dfbfd7a44d486f6563125f2a4d6b15878361e507cf11ae6

data/CHANGELOG.md

@@ -10,7 +10,12 @@ This CHANGELOG follows the format listed at [Keep A Changelog](http://keepachang
 ### Fixed
 ### Changed
 
-## [Unreleased]
+## [0.8.1]
+
+### Fixed
+- check-consumer-lag: remove celluloid (temporary), fix data structure, fix partition_owner function
+
+## [0.8.0]
 
 ### Breaking Changes
 - Refactoring check-consumer-lag to remove shell/jvm execution

data/bin/check-consumer-lag.rb

@@ -27,17 +27,10 @@
 
 require 'sensu-plugin/check/cli'
 
-require 'celluloid-io'
 require 'json'
 require 'poseidon'
 require 'zookeeper'
 
-module Poseidon
-  class Connection
-    TCPSocket = Celluloid::IO::TCPSocket
-  end
-end
-
 class ConsumerLagCheck < Sensu::Plugin::Check::CLI
   option :group,
          description: 'Consumer group',
@@ -89,8 +82,8 @@ class ConsumerLagCheck < Sensu::Plugin::Check::CLI
     zk.get(path: "/consumers/#{group}/offsets/#{topic}/#{partition}")[:data].to_i
   end
 
-  def partition_owner(zk, group, partition)
-    zk.get(path: "/consumers/#{group}/offsets/#{owners}/#{partition}")[:data]
+  def partition_owner(zk, group, topic, partition)
+    zk.get(path: "/consumers/#{group}/owners/#{topic}/#{partition}")[:data]
   end
 
   def run
@@ -103,10 +96,10 @@ class ConsumerLagCheck < Sensu::Plugin::Check::CLI
 
     consumers = {}
     topics.each do |topic|
-      consumers[topic] = {}
+      consumers[topic] = []
 
       topics_partitions(z, topic).each do |partition|
-        owner = partition_owner(z, group, partition)
+        owner = partition_owner(z, group, topic, partition)
 
         leader = leader_broker(z, topic, partition)
         consumer = Poseidon::PartitionConsumer.new('CheckConsumerLag', leader['host'], leader['port'], topic, partition, :latest_offset)
@@ -115,27 +108,22 @@ class ConsumerLagCheck < Sensu::Plugin::Check::CLI
         offset = consumer_offset(z, group, topic, partition)
 
         lag = logsize - offset
-
-        consumers[topic][:partition] = partition
-        consumers[topic][:logsize] = logsize
-        consumers[topic][:offset] = offset
-        consumers[topic][:lag] = lag
-        consumers[topic][:owner] = owner
+        consumers[topic].push(partition: partition, logsize: logsize, offset: offset, lag: lag, owner: owner)
       end
     end
 
     [:offset, :logsize, :lag].each do |field|
-      consumers.map do |k, v|
-        critical "Topic #{k} has partitions with #{field} < 0" unless v.select { |w| w[field].to_i < 0 }.empty?
+      consumers.each do |k, v|
+        critical "Topic #{k} has #{field} < 0 '#{v[field]}'" unless v.select { |w| w[field].to_i < 0 }.empty?
       end
     end
 
-    consumers.map do |k, v|
-      critical "Topic #{k} has partitions with no owner" unless v.select { |w| w[:owner] == 'none' }.empty?
+    consumers.each do |k, v|
+      critical "Topic #{k} has partitions with no owner '#{v[:owner]}'" unless v.select { |w| w[:owner] == 'none' }.empty?
     end
 
-    lags = topics.map do |k, v|
-      Hash[k, v.inject(0) { |a, e| a + e[:lag].to_i }]
+    lags = consumers.map do |k, v|
+      Hash[k, v.inject(0) { |a, e| a + e[:lag] }]
     end
 
     max_lag = lags.map(&:values).flatten.max

data/lib/sensu-plugins-kafka/version.rb

@@ -6,7 +6,7 @@ module SensuPluginsKafka
   module Version
     MAJOR = 0
     MINOR = 8
-    PATCH = 0
+    PATCH = 1
 
     VER_STRING = [MAJOR, MINOR, PATCH].compact.join('.')
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sensu-plugins-kafka
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Sensu-Plugins and contributors
@@ -9,22 +9,8 @@ autorequire:
 bindir: bin
 cert_chain:
 - certs/sensu-plugins.pem
-date: 2017-02-15 00:00:00.000000000 Z
+date: 2017-02-16 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: celluloid-io
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.17.3
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.17.3
 - !ruby/object:Gem::Dependency
   name: poseidon
   requirement: !ruby/object:Gem::Requirement