Fixing issue redis#4592 by loading node info before running move_slot…

…, which will crash if the node in question has become a slave
tsein-bc · Oct 9, 2018 · 78e1cb4 · 78e1cb4
1 parent c7613cf
commit 78e1cb4
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/redis-trib.rb b/src/redis-trib.rb
@@ -1084,6 +1084,8 @@ def move_slot(source,target,slot,o={})
         # Set the new node as the owner of the slot in all the known nodes.
         if !o[:cold]
             @nodes.each{|n|
+                # Load node info in case this node became a slave before the slot could be moved.
+                n.load_info
                 next if n.has_flag?("slave")
                 n.r.cluster("setslot",slot,"node",target.info[:name])
             }