fixme

Author: madhukar93

Diff for: server/src/main/java/com/cloud/network/NetworkServiceImpl.java

@@ -4997,6 +4997,7 @@ public PhysicalNetworkTrafficType addTrafficTypeToPhysicalNetwork(Long physicalN
             if (TrafficType.Public.equals(trafficType)) {
                 List<String> isolationMethods = network.getIsolationMethods();
                 if ((isolationMethods.size() == 1 && isolationMethods.get(0).toLowerCase().equals("vxlan"))
+                        // TODO: what happens if there are multiple isolation methods, why are multiple isolation methods allowed per pnet? They seem to be filterable by provider/preifx (PhysicalNetwork.java). Documentation says only two are allowed but code and tests contradict this.
                         || (isolationMethod != null && isolationMethods.contains(isolationMethod) && isolationMethod.toLowerCase().equals("vxlan"))) {
                     // find row in networks table that is defined as 'Public', created when zone was deployed
                     NetworkVO publicNetwork = _networksDao.listByZoneAndTrafficType(network.getDataCenterId(), TrafficType.Public).get(0);