madhukar93 commented on code in PR #8151:
URL: https://github.com/apache/cloudstack/pull/8151#discussion_r1374223984
##########
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.
Review Comment:
> Yes, that would be okay.
to summarize - even though string array is getting serialized correctly (cmk
output in description), I will stick to how it is handled currently in the
[PhysicalNetworkResponse](https://github.com/apache/cloudstack/blob/a06f8a8763dc0d30b14e41d579ec73d40613e1d9/api/src/main/java/org/apache/cloudstack/api/response/PhysicalNetworkResponse.java#L129)
where the methods array is converted into a string csv value.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
