----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11380/#review21185 -----------------------------------------------------------
Commit 965c7b9c35fc5680b2f35a8c8fed35c1232a643b in branch refs/heads/master from Prachi Damle <pra...@cloud.com> [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=965c7b9 ] Patch 2: CLOUDSTACK-681: Dedicated Resources - Explicit Dedication, Private zone, pod, cluster or host Patch 2 for https://reviews.apache.org/r/11379/ Created for files server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java, server/test/com/cloud/vm/DeploymentPlanningManagerImplTest.java, server/test/org/apache/cloudstack/affinity/AffinityApiUnitTest.java - ASF Subversion and Git Services On May 30, 2013, 7:23 a.m., Saksham Srivastava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11380/ > ----------------------------------------------------------- > > (Updated May 30, 2013, 7:23 a.m.) > > > Review request for cloudstack, Devdeep Singh and Prachi Damle. > > > Description > ------- > > Patch 2 for https://reviews.apache.org/r/11379/ > Created for files > server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java, > server/test/com/cloud/vm/DeploymentPlanningManagerImplTest.java, > server/test/org/apache/cloudstack/affinity/AffinityApiUnitTest.java > > > This addresses bug CLOUDSTACK-681. > > > Diffs > ----- > > server/src/com/cloud/configuration/ConfigurationManagerImpl.java 79375f9 > server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java 795b526 > server/test/com/cloud/vm/DeploymentPlanningManagerImplTest.java e3b7d31 > server/test/org/apache/cloudstack/affinity/AffinityApiUnitTest.java 24c5d3d > > Diff: https://reviews.apache.org/r/11380/diff/ > > > Testing > ------- > > Unittest: For the new APIs and Service, added unit tests under : > plugins/dedicated-resources/test/org/apache/cloudstack/dedicated/DedicatedApiUnitTest.java > Marvin Test: To dedicate host, create affinity group, deploy-vm, check if vm > is deployed on the dedicated host. > Rat Build Successful. > > > Thanks, > > Saksham Srivastava > >