----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13627/#review25402 -----------------------------------------------------------
Commit 5528ba4b20593e1c883507a1e161ccb48fffa57b in branch refs/heads/4.2-forward from Koushik Das [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5528ba4 ] CLOUDSTACK-3441: [Load Test] High delays between VM being allocated to Pod and network implementation causing delays in VM deployment The locking code in implement/shutdown network code was not efficient. Even in order to check the current state of the network lock was getting acquired which is not required. This resulted in delays in deploy VM as can be seen from attached logs where the code waited on the lock just to check if network is implemented. As part of the fix moved out code that is checking if the network is already implemented or shutdowned outside the lock. - ASF Subversion and Git Services On Aug. 17, 2013, 9:32 a.m., Koushik Das wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13627/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2013, 9:32 a.m.) > > > Review request for cloudstack, Alex Huang, Abhinandan Prateek, Chiradeep > Vittal, and Murali Reddy. > > > Bugs: CLOUDSTACK-3441 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-3441: [Load Test] High delays between VM being allocated to Pod > and network implementation causing delays in VM deployment > > The locking code in implement/shutdown network code was not efficient. Even > in order to check the current state of the network lock was getting acquired > which is not required. This resulted in delays in deploy VM as can be seen > from attached logs where the code waited on the lock just to check if network > is implemented. > > As part of the fix moved out code that is checking if the network is already > implemented or shutdowned outside the lock. > > > Diffs > ----- > > server/src/com/cloud/network/NetworkManagerImpl.java 68b1b4f > > Diff: https://reviews.apache.org/r/13627/diff/ > > > Testing > ------- > > > Thanks, > > Koushik Das > >
