----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13188/#review24418 -----------------------------------------------------------
Ship it! server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java <https://reviews.apache.org/r/13188/#comment48323> Ideally we should make it part of the constructor. - Nitin Mehta On Aug. 1, 2013, 6:35 a.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13188/ > ----------------------------------------------------------- > > (Updated Aug. 1, 2013, 6:35 a.m.) > > > Review request for cloudstack and Nitin Mehta. > > > Bugs: CLOUDSTACK-3042 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-3042: handle System VM Scaling up of memory/CPU based on the > presence of tools in the template > > > Diffs > ----- > > server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java 5983aa7 > server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java > 3ee8a1c > server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java > 6859b0b > > Diff: https://reviews.apache.org/r/13188/diff/ > > > Testing > ------- > > Tested locally > > > Thanks, > > Harikrishna Patnala > >
