----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10022/#review18119 -----------------------------------------------------------
Commit 0ce1d4fde1fb9e68ed83a9e094133701abbee291 in branch refs/heads/4.1 from Chip Childers <chip.child...@gmail.com> [ https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;h=0ce1d4f ] CLOUDSTACK-1668: Fix IP conflict in VPC tier Currently, allPossibleIps return the Ip lists which include the gateway, so we need to remove gateway ip from this list. Now, for non-VPC network it works, because NetUtils.getAllIpsFromCidr return the Ip lists which do not include the first IP of the network (like 192.168.0.1). We need too add the first IP into the returned Ip list, because it can be used for VM if it is not the gateway IP (for example, VPC networks). The corresponding patch for 4.0.1 has been posted on https://reviews.apache.org/r/9923/ Signed-off-by: Chip Childers <chip.child...@gmail.com> - ASF Subversion and Git Services On March 19, 2013, 8:25 p.m., Wei Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10022/ > ----------------------------------------------------------- > > (Updated March 19, 2013, 8:25 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > Currently, allPossibleIps return the Ip lists which include the gateway, so > we need to remove gateway ip from this list. > > Now, for non-VPC network it works, because NetUtils.getAllIpsFromCidr return > the Ip lists which do not include the first IP of the network (like > 192.168.0.1). > We need too add the first IP into the returned Ip list, because it can be > used for VM if it is not the gateway IP (for example, VPC networks). > > The corresponding patch for 4.0.1 has been posted on > https://reviews.apache.org/r/9923/ > > > This addresses bug CLOUDSTACK-1668. > > > Diffs > ----- > > server/src/com/cloud/network/NetworkModelImpl.java 779b9f2 > server/src/com/cloud/network/NetworkServiceImpl.java 52e81e5 > utils/src/com/cloud/utils/net/NetUtils.java dd40a33 > > Diff: https://reviews.apache.org/r/10022/diff/ > > > Testing > ------- > > > Thanks, > > Wei Zhou > >
