----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11380/#review21179 -----------------------------------------------------------
Ship it! Ship It! - Prachi Damle On May 28, 2013, 1:07 p.m., Saksham Srivastava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11380/ > ----------------------------------------------------------- > > (Updated May 28, 2013, 1:07 p.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/deploy/DeploymentPlanningManagerImpl.java 795b526 > server/test/com/cloud/vm/DeploymentPlanningManagerImplTest.java e3b7d31 > server/test/org/apache/cloudstack/affinity/AffinityApiUnitTest.java 484b044 > > 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 > >
