Skip to content

Commit e8b249f

Browse files
authored
server: fix netowork upgrade for IPv6 (#6729)
Fixes network update for IPv6 offering upgrade
1 parent a45a4a5 commit e8b249f

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

server/src/main/java/com/cloud/network/Ipv6ServiceImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -501,7 +501,7 @@ public void checkNetworkIpv6Upgrade(Network network) throws InsufficientAddressC
501501
ipAddressDao.listByAssociatedVpc(network.getVpcId(), true);
502502
for (IPAddressVO address : addresses) {
503503
VlanVO vlan = vlanDao.findById(address.getVlanId());
504-
final List<VlanVO> ranges = vlanDao.listIpv6RangeByZoneIdAndVlanId(network.getPhysicalNetworkId(), vlan.getVlanTag());
504+
final List<VlanVO> ranges = vlanDao.listIpv6RangeByZoneIdAndVlanId(network.getDataCenterId(), vlan.getVlanTag());
505505
if (CollectionUtils.isEmpty(ranges)) {
506506
s_logger.error(String.format("Unable to find IPv6 address for zone ID: %d, physical network ID: %d, VLAN: %s", network.getDataCenterId(), network.getPhysicalNetworkId(), vlan.getVlanTag()));
507507
InsufficientAddressCapacityException ex = new InsufficientAddressCapacityException("Insufficient address capacity", DataCenter.class, network.getDataCenterId());

server/src/test/java/com/cloud/network/Ipv6ServiceImplTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -592,10 +592,10 @@ public void testCheckNetworkIpv6UpgradeForNoPrefixes() {
592592

593593
@Test
594594
public void testCheckNetworkIpv6UpgradeForNoIpv6Vlan() {
595-
final long physicalNetworkId = 1L;
595+
final long zoneId = 1L;
596596
Mockito.when(dataCenterGuestIpv6PrefixDao.listByDataCenterId(Mockito.anyLong())).thenReturn(List.of(Mockito.mock(DataCenterGuestIpv6PrefixVO.class)));
597597
Network network = Mockito.mock(Network.class);
598-
Mockito.when(network.getPhysicalNetworkId()).thenReturn(physicalNetworkId);
598+
Mockito.when(network.getDataCenterId()).thenReturn(zoneId);
599599
Mockito.when(network.getVpcId()).thenReturn(null);
600600
Mockito.when(ipAddressDao.listByAssociatedNetwork(Mockito.anyLong(), Mockito.anyBoolean())).thenReturn(List.of(Mockito.mock(IPAddressVO.class)));
601601
VlanVO vlanVO = Mockito.mock(VlanVO.class);
@@ -610,16 +610,16 @@ public void testCheckNetworkIpv6UpgradeForNoIpv6Vlan() {
610610

611611
@Test
612612
public void testCheckNetworkIpv6UpgradeForNetwork() {
613-
final long physicalNetworkId = 1L;
613+
final long zoneId = 1L;
614614
Mockito.when(dataCenterGuestIpv6PrefixDao.listByDataCenterId(Mockito.anyLong())).thenReturn(List.of(Mockito.mock(DataCenterGuestIpv6PrefixVO.class)));
615615
Network network = Mockito.mock(Network.class);
616-
Mockito.when(network.getPhysicalNetworkId()).thenReturn(physicalNetworkId);
616+
Mockito.when(network.getDataCenterId()).thenReturn(zoneId);
617617
Mockito.when(network.getVpcId()).thenReturn(null);
618618
Mockito.when(ipAddressDao.listByAssociatedNetwork(Mockito.anyLong(), Mockito.anyBoolean())).thenReturn(List.of(Mockito.mock(IPAddressVO.class)));
619619
VlanVO vlanVO = Mockito.mock(VlanVO.class);
620620
Mockito.when(vlanVO.getVlanTag()).thenReturn(vlan);
621621
Mockito.when(vlanDao.findById(Mockito.anyLong())).thenReturn(vlanVO);
622-
Mockito.when(vlanDao.listIpv6RangeByZoneIdAndVlanId(physicalNetworkId, vlan)).thenReturn(List.of(vlanVO));
622+
Mockito.when(vlanDao.listIpv6RangeByZoneIdAndVlanId(zoneId, vlan)).thenReturn(List.of(vlanVO));
623623
try {
624624
ipv6Service.checkNetworkIpv6Upgrade(network);
625625
} catch (InsufficientAddressCapacityException | ResourceAllocationException e) {

0 commit comments

Comments
 (0)