review comments

Author: admin

.wlock

@@ -64,7 +64,7 @@
         {
             "name": "podIpRange",
             "label": "Pod IP range",
-            "description": "Range name or CIDR block/mask notation, e.g. 10.0.0.0/21 or /21",
+            "description": "Range name or CIDR block/mask notation, e.g. 10.0.0.0/21 or /21. If using a named range, ensure it is larger than /21.",
             "type": "STRING",
             "mandatory": false,
             "visibilityCondition": "model.isVpcNative"

python-clusters/create-gke-cluster/cluster.json

@@ -211,6 +211,8 @@ def with_labels(self, labels={}):
     def with_vpc_native_settings(self, is_vpc_native, pod_ip_range, svc_ip_range):
         if is_vpc_native:
             self.is_vpc_native = is_vpc_native
+            if pod_ip_range is not None and len(pod_ip_range) > 0 and pod_ip_range == svc_ip_range:
+                raise Exception("Service IP range must be different from pod IP range")
             self.pod_ip_range = pod_ip_range
             self.svc_ip_range = svc_ip_range
         return self