kazoo-ruby 0.5.6 → 0.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 75139fef3e6343c067f6a34f6a5e5c520be5693c
-  data.tar.gz: dff9de16690c46259bdec390215dfa5bffcff5dc
+  metadata.gz: cddff6bb622ee9d1db1980bda5a00eef2c6f1359
+  data.tar.gz: 5ac18d330d8f2da8136dab9b583e60fb40dda1e0
 SHA512:
-  metadata.gz: 12b210d80021ba4fb9beb09fb775f82a0d35cc685365b2b9f0c1ee9222ba0a8f8c59ad0c4e3a3b5c96f5f9eb1a5cdb336977cd375db97f3b7f837e1670bd8ac5
-  data.tar.gz: 4991813165f7b3938026ca02668239ea87e724ceae01dcd5d056cfab14d2c36434cbf1ece99c579f90c9e7019ce10b077bc60aad9a624adb9079e204d448e4fe
+  metadata.gz: e4656dd463c13c37dc92da902e77c7ed275d3a3aa1a9a934c03c54a0cd176ea2a4e0061fad18fc2552ad8296fb374d1dce385e6cced19a80acf2a24035948300
+  data.tar.gz: 4ff31ca4aa664b76fe5af430e2a0a041d6d74f13b14251004c4511bdb945349cc49361c66a22e7d1c74499afa6d31459a5a61f7cf7366ab650a5b9ab7ad99f97

data/lib/kazoo/partition.rb

@@ -98,7 +98,12 @@ module Kazoo
       json = JSON.parse(json_payload)
       raise Kazoo::VersionNotSupported unless json.fetch('version') == 1
 
-      @leader = cluster.brokers.fetch(json.fetch('leader'))
+      begin
+        @leader = cluster.brokers.fetch(json.fetch('leader'))
+      rescue KeyError
+        raise Kazoo::Error, "Topic #{topic.name}:#{id} has unknown leader! Brokers: '#{cluster.brokers.keys.join(',')}'; 'Payload: '#{json_payload.inspect}'"
+      end
+
       @isr = json.fetch('isr').map do |r|
         begin
           cluster.brokers.fetch(r)

data/lib/kazoo/version.rb

@@ -1,3 +1,3 @@
 module Kazoo
-  VERSION = "0.5.6"
+  VERSION = "0.5.7"
 end

data/test/partition_test.rb

@@ -55,4 +55,15 @@ class PartitionTest < Minitest::Test
 
     assert_raises(Kazoo::Error) { partition.under_replicated? }
   end
+
+  def test_raises_unknown_leader
+    partition = @cluster.topics['test.1'].partitions[0]
+    partition.unstub(:leader)
+    partition.unstub(:isr)
+
+    json_payload = '{"controller_epoch":90,"leader":-1,"version":1,"leader_epoch":135,"isr":[2,1,0]}'
+    @cluster.zk.expects(:get).with(path: "/brokers/topics/test.1/partitions/0/state").returns(data: json_payload, rc: 0)
+
+    assert_raises(Kazoo::Error) { partition.under_replicated? }
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kazoo-ruby
 version: !ruby/object:Gem::Version
-  version: 0.5.6
+  version: 0.5.7
 platform: ruby
 authors:
 - Willem van Bergen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-19 00:00:00.000000000 Z
+date: 2019-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor