Skip to content

Commit 823a789

Browse files
widoyadvr
authored andcommitted
CLOUDSTACK-10220: Configure IPv4 alias on VR regardless of IPv6 (#2396)
IPv4 and IPv6 are two different protocols and the presence of IPv6 in a network does not mean that IPv4 aliases/multiple subnets should not be configured or supported by the VR. This if-statement was written almost 5 years ago in a attempt to add IPv6 support to CloudStack but was never fully implemented. Signed-off-by: Wido den Hollander <wido@widodh.nl>
1 parent d84b268 commit 823a789

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1306,9 +1306,7 @@ protected boolean prepareElement(final NetworkElement element, final Network net
13061306
if (_networkModel.areServicesSupportedInNetwork(network.getId(), Service.Dhcp)
13071307
&& _networkModel.isProviderSupportServiceInNetwork(network.getId(), Service.Dhcp, element.getProvider()) && element instanceof DhcpServiceProvider) {
13081308
final DhcpServiceProvider sp = (DhcpServiceProvider)element;
1309-
final Map<Capability, String> dhcpCapabilities = element.getCapabilities().get(Service.Dhcp);
1310-
final String supportsMultipleSubnets = dhcpCapabilities.get(Capability.DhcpAccrossMultipleSubnets);
1311-
if (supportsMultipleSubnets != null && Boolean.valueOf(supportsMultipleSubnets) && profile.getIPv6Address() == null) {
1309+
if (isDhcpAccrossMultipleSubnetsSupported(sp)) {
13121310
if (!sp.configDhcpSupportForSubnet(network, profile, vmProfile, dest, context)) {
13131311
return false;
13141312
}

0 commit comments

Comments
 (0)