----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11341/#review21788 -----------------------------------------------------------
Ship it! Ship It! - Abhinandan Prateek On May 23, 2013, 9:45 a.m., venkata swamy babu budumuru wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11341/ > ----------------------------------------------------------- > > (Updated May 23, 2013, 9:45 a.m.) > > > Review request for cloudstack and Abhinandan Prateek. > > > Description > ------- > > Steps to reproduce : > > 1. Have a CloudStack 4.2 setup > 2. Try to create an advanced zone using the XenServer 6.2.0 > > Observations : > > (i) Adding host fails because XcpDiscoverer.java is currently allowing only > till XenServer 6.1.0 release. > > > This addresses bug CLOUDSTACK-2642. > > > Diffs > ----- > > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/discoverer/XcpServerDiscoverer.java > fd49836 > > Diff: https://reviews.apache.org/r/11341/diff/ > > > Testing > ------- > > With the above mentioned fix, tried to add the XenServer 6.2 build and it > went fine without any issues. > > > Thanks, > > venkata swamy babu budumuru > >
