Instance deregistration should try all associated ELBs. Fixes #869

2015-02-27 01:05:47 -08:00 · 2015-02-27 01:05:47 -08:00 · 9de952cb3f
commit 9de952cb3f
parent eff6307ac3
1 changed files with 3 additions and 3 deletions
--- a/lib/ansible/modules/cloud/amazon/ec2_elb.py
+++ b/lib/ansible/modules/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])