GabrielBrascher commented on a change in pull request #4177:
URL: https://github.com/apache/cloudstack/pull/4177#discussion_r446148602
##########
File path:
server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java
##########
@@ -3781,7 +3781,7 @@ public Vlan createVlanAndPublicIpRange(final long zoneId,
final long networkId,
ipv4 = true;
}
- if (startIPv6 != null) {
+ if (vlanIp6Cidr != null) {
Review comment:
CloudStack uses SLAAC for managing IPv6 ranges, therefore it is not
necessary to have a start/end IPv6 address; on the other hand, IPv6 CIDR is
mandatory for IPv6 networks.
That is why I changed this and other pieces of code that had start/end ipv6
address as mandatory.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
