----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12808/#review23627 -----------------------------------------------------------
Review 12808 PASSED the build test The url of build cloudstack-master-with-patch #10 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/10/ - Jenkins Cloudstack.org On July 22, 2013, 11:03 a.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12808/ > ----------------------------------------------------------- > > (Updated July 22, 2013, 11:03 a.m.) > > > Review request for cloudstack, Koushik Das and Nitin Mehta. > > > Bugs: CLOUDSTACK-2536 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and > (cpu/memory).overprosioning.factor are redundant(cluster level) > There are two APIs updateCluster and UpdateConfiguration where we can update > these parameters. Ideally this should get updated using updateConfiguration > API, so removing from updateCluster API > > > Diffs > ----- > > api/src/com/cloud/resource/ResourceService.java 25298cc > > api/src/org/apache/cloudstack/api/command/admin/cluster/UpdateClusterCmd.java > a14f905 > server/src/com/cloud/configuration/ConfigurationManagerImpl.java 92178c9 > server/src/com/cloud/resource/ResourceManagerImpl.java d7652f4 > server/test/com/cloud/resource/MockResourceManagerImpl.java 2c66134 > > Diff: https://reviews.apache.org/r/12808/diff/ > > > Testing > ------- > > > Thanks, > > Harikrishna Patnala > >
