----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8568/#review14434 -----------------------------------------------------------
Ship it! Committed to master, please close this review. commit 2cc09c813a419ab7f81796b79abde849eb456698 Author: Devdeep Singh <devdeep.si...@citrix.com> Date: Thu Dec 13 09:42:43 2012 -0500 CLOUDSTACK-594: Updating the XenServer java bindings used in cloudstack - part 3. Updating the xenserver java bindings used in cloudstack. This is part 3 of the change and it adds the customization's that have been made to them for cloudstack. 1. https://reviews.apache.org/r/8566/ - removes the old bindings. 2. https://reviews.apache.org/r/8567/ - introduces the latest bindings. 3. https://reviews.apache.org/r/8568/ - adds the customization's made to them for cloudstack. Signed-off-by: Chip Childers <chip.child...@gmail.com> - Chip Childers On Dec. 13, 2012, 8:38 a.m., Devdeep Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8568/ > ----------------------------------------------------------- > > (Updated Dec. 13, 2012, 8:38 a.m.) > > > Review request for cloudstack, Abhinandan Prateek and anthony xu. > > > Description > ------- > > Updating the xenserver java bindings used in cloudstack. This is part 3 of > the change and it adds the customization's that have been made to them for > cloudstack. There are three review requests submitted for updating the > bindings. Kindly review and apply them in the following order > 1. https://reviews.apache.org/r/8566/ - removes the old bindings. > 2. https://reviews.apache.org/r/8567/ - introduces the latest bindings. > 3. https://reviews.apache.org/r/8568/ - adds the customization's made to them > for cloudstack. > > > This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-594. > > > Diffs > ----- > > deps/XenServerJava/src/com/xensource/xenapi/Connection.java PRE-CREATION > > Diff: https://reviews.apache.org/r/8568/diff/ > > > Testing > ------- > > Did the following tests for both XenServer 5.6 and XenServer 6.1 > 1. Did a setup and created a cluster of XenServer hosts. > 2. Deployed instances. Stopped and started them. > 3. Migration of instances with in the cluster. > > > Thanks, > > Devdeep Singh > >