From: Alfredo Deza <alfredo.deza@inktank.com>
Date: Wed, 27 Aug 2014 19:19:57 +0000 (-0400)
Subject: host validation ip should be done per subnet, not per ip
X-Git-Tag: v1.5.13~6
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=3f77d5177d1c80f83a6ba434b81f8bfcd7fd5cf7;p=ceph-deploy.git

host validation ip should be done per subnet, not per ip

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
---

diff --git a/ceph_deploy/new.py b/ceph_deploy/new.py
index 2e24405..d87c3ec 100644
--- a/ceph_deploy/new.py
+++ b/ceph_deploy/new.py
@@ -77,29 +77,28 @@ def ssh_copy_keys(hostname, username=None):
 
 def validate_host_ip(ips, subnets):
     """
-    Make sure that a given host will have IP addresses that will be
-    present in one (or all) of the the subnets specified
+    Make sure that a given host all subnets specified will have at least one IP
+    in that range.
     """
     # Make sure we prune ``None`` arguments
     subnets = [s for s in subnets if s is not None]
     validate_one_subnet = len(subnets) == 1
 
-    def ip_in_one_subnet(ip, subnets):
+    def ip_in_one_subnet(ips, subnet):
         """ ensure an ip exists in at least one subnet """
-        for subnet in subnets:
-            if subnet:
-                if net.ip_in_subnet(ip, subnet):
-                    return True
+        for ip in ips:
+            if net.ip_in_subnet(ip, subnet):
+                return True
         return False
 
-    for ip in ips:
-        if ip_in_one_subnet(ip, subnets):
+    for subnet in subnets:
+        if ip_in_one_subnet(ips, subnet):
             if validate_one_subnet:
                 return
             else:  # keep going to make sure the other subnets are ok
                 continue
         else:
-            msg = "IP (%s) is not valid for any of the subnets specified %s" % (ip, str(subnets))
+            msg = "subnet (%s) is not valid for any of the ips found %s" % (subnet, str(ips))
             raise RuntimeError(msg)