----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9839/#review17690 -----------------------------------------------------------
Ship it! Applied to 4.1 (I'll push to master as well): commit 29ec31c645cbf0fce73f4818a9b53e836d7e6835 Author: Dennis Lawler <dlaw...@gmail.com> Date: Sun Mar 10 12:16:37 2013 -0700 CLOUDSTACK-1620: getXenServer602GuestOsType -> getXenServer610GuestOsType Fixing XenServer 6.1 OS types incompatible with XS6.0 Thanks for the patch! - Chip Childers On March 11, 2013, 3:11 a.m., Dennis Lawler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9839/ > ----------------------------------------------------------- > > (Updated March 11, 2013, 3:11 a.m.) > > > Review request for cloudstack. > > > Description > ------- > > Current trunk Cloudstack does not provision CentOS 6 guest OS type. > This is because of a minor copy-paste error and an incorrect Guest OS > template list. > While I was at it, I fixed up a few other template types to match the current > XenServer 6.1 template list. > > > This addresses bug CLOUDSTACK-1620. > > > Diffs > ----- > > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java > 1b8496f > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer610Resource.java > 17ad3e6 > > Diff: https://reviews.apache.org/r/9839/diff/ > > > Testing > ------- > > Provisioned CentOS 6 and Ubuntu 12.04 VMs on XS 6.1. > > > Thanks, > > Dennis Lawler > >