Re: Review Request 12264: CLOUDSTACK-3351: Fix add cluster API to set right hypervisor type

2013-08-01 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12264/#review24425 --- Commit b4fe0252230e74d36f62679cd2a751f278edacc1 in branch

Review Request 12264: CLOUDSTACK-3351: Fix add cluster API to set right hypervisor type

2013-07-04 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12264/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: CLOUDSTACK-3351

Re: Review Request 12264: CLOUDSTACK-3351: Fix add cluster API to set right hypervisor type

2013-07-04 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12264/#review22759 --- Since we have already put it to be case insensitive will this be

Re: Review Request 12264: CLOUDSTACK-3351: Fix add cluster API to set right hypervisor type

2013-07-04 Thread Harikrishna Patnala
On July 4, 2013, 9:10 a.m., Nitin Mehta wrote: Since we have already put it to be case insensitive will this be causing issues for upgrade or anywhere else we are comparing. This is concerning As long as we get hypervisor type from clusterVO there is no problem since VO converts it to