cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Saksham Srivastava" <saksham.srivast...@citrix.com>
Subject Re: Review Request: Patch 2: CLOUDSTACK-681: Dedicated Resources - Explicit Dedication, Private zone, pod, cluster or host
Date Thu, 30 May 2013 07:23:08 GMT

-----------------------------------------------------------
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.


Changes
-------

Resolving merge conflicts.


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 (updated)
-----

  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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message