Instance deregistration should try all associated ELBs. Fixes #869

10 years ago · 07d562c58a
parent b0c534842b
commit 07d562c58a
1 changed files with 3 additions and 3 deletions
--- a/cloud/amazon/ec2_elb.py
+++ b/cloud/amazon/ec2_elb.py
@ -130,9 +130,9 @@ class ElbManager:
        for lb in self.lbs:
            initial_state = self._get_instance_health(lb) 
            if initial_state is None:
-                # The instance isn't registered with this ELB so just 
-                # return unchanged
-                return
+                # Instance isn't registered with this load
+                # balancer. Ignore it and try the next one.
+                continue

            lb.deregister_instances([self.instance_id])