----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17736/#review37061 -----------------------------------------------------------
server/src/com/cloud/vm/UserVmManagerImpl.java <https://reviews.apache.org/r/17736/#comment68317> Two queries are executed in this for loop which puts stress on the database and slows down the application. Furthermore, this for loop appears to be performing in-memory join and filter operations. Please refactor into a single query performed by the most appropriate DAO. - John Burwell On March 13, 2014, 3:31 a.m., Saksham Srivastava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17736/ > ----------------------------------------------------------- > > (Updated March 13, 2014, 3:31 a.m.) > > > Review request for cloudstack, John Burwell and Murali Reddy. > > > Bugs: CLOUDSTACK-5999 > https://issues.apache.org/jira/browse/CLOUDSTACK-5999 > > > Repository: cloudstack-git > > > Description > ------- > > When a guest is rebooted from CloudStack, if the virtual router managing the > guest network of that guest is down, CloudStack will not start the virtual > router. > However the router is started in case the guest vm is stopped and then > started. > To mantain similarity between the 2 process it is necessary to start the VR > in case it is not running. > The fix will address the same. > > > Diffs > ----- > > server/src/com/cloud/vm/UserVmManagerImpl.java be00aa8 > > Diff: https://reviews.apache.org/r/17736/diff/ > > > Testing > ------- > > Testing: > 1) vm in a single guest network : > vm Reboot : If the VR is stopped: VR is first started and then > the VM reboots. > vm Reboot : If the VR is running, VM reboots as it used to. > vm Stop/Start continue to work the same. > > 2) vm having nics in multi networks : > vm Reboot : If VR in any/all network is stopped: VRs are first > started and then the VM reboots. > vm Reboot : If the VRs are running, VM reboots as it used to. > vm Stop/Start continue to work the same. > > 3) vpc : > Tested the above scenarios for vpc also, works fine. > > Patch applies cleanly. > Build passes succesfully. > FindBug is passed. > Rebased against latest master. > > > Thanks, > > Saksham Srivastava > >