----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11341/#review20935 -----------------------------------------------------------
Thanks for this but probably needs more testing - maybe even running the BVT against this. I see that CitrixHelper should be changed for this...maybe more places as well, depending on what has changed in this version - Nitin Mehta 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 > >
