----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12849/#review25215 -----------------------------------------------------------
Ship it! With no open issues and all review items processed i'm shipping this patch. - Hugo Trippaers On Aug. 15, 2013, 7:53 a.m., daan Hoogland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12849/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2013, 7:53 a.m.) > > > Review request for cloudstack, Chiradeep Vittal, Dave Cahill, Koushik Das, > Hugo Trippaers, and Sheng Yang. > > > Bugs: CLOUDSTACK-4345 > > > Repository: cloudstack-git > > > Description > ------- > > Both BroadcastDomainType and IsolationType needed some extra code for > backwards compatibility. > > All over the code calls are done to URI.getHost() to retrieve ids of > broadcastdomains. These id obviously are not hosts so this call is confusing > and requires maintenance all over the code base. Also for different types the > value returned by getHost has different meaning. vlan://1 is id 1 of course > but others might be ranges of vlans or colon separated values. To make things > worse a NiciraNvp has an uri of the form lswitch:<uuid> without the forward > slashes. > > To make the system more maintainable in this perspect the changes in this > patch were made. It is my intention to replace the calls to getHost by the > member call getValueFrom or the static method getValue in time. In this way > maintenance is centralized and an overview of differnces and quirks is easily > found > > > Diffs > ----- > > api/src/com/cloud/network/Networks.java c76c3d4 > api/test/com/cloud/network/NetworksTest.java 31114e8 > > Diff: https://reviews.apache.org/r/12849/diff/ > > > Testing > ------- > > unit tests for different kind of BroadcastDomainType.values > > > Thanks, > > daan Hoogland > >