cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nitin Mehta" <nitin.me...@citrix.com>
Subject Re: Review Request: Fix for CS-15631
Date Tue, 31 Jul 2012 13:06:17 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6229/#review9641
-----------------------------------------------------------

Ship it!


Ship It!

- Nitin Mehta


On July 31, 2012, 1 p.m., Koushik Das wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6229/
> -----------------------------------------------------------
> 
> (Updated July 31, 2012, 1 p.m.)
> 
> 
> Review request for cloudstack and Abhinandan Prateek.
> 
> 
> Description
> -------
> 
> Support for up to 16 VDIs per VM on XS 6.0 and above (16 VDIs => root + cd + 14 data
volumes). Currently in CS number of data disk that can be attached to VM is hard-coded to
6. Made this setting configurable by moving it to hypervisor capabilities. Although XS 6.0
and above supports upto 16 VDIs but while testing on XS 6.0.2 found that only 13 data volumes
can be attached to a VM. So for XS 6.0 and 6.0.2 max_data_volumes_limit is set to 13 currently.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/hypervisor/HypervisorCapabilities.java d0fcf01 
>   core/src/com/cloud/hypervisor/HypervisorCapabilitiesVO.java 33cafbc 
>   server/src/com/cloud/hypervisor/dao/HypervisorCapabilitiesDao.java c7fb969 
>   server/src/com/cloud/hypervisor/dao/HypervisorCapabilitiesDaoImpl.java 8feeba5 
>   server/src/com/cloud/vm/UserVmManagerImpl.java 649e215 
>   setup/db/create-schema.sql ae5f581 
>   setup/db/db/schema-303to40.sql 01710b1 
> 
> Diff: https://reviews.apache.org/r/6229/diff/
> 
> 
> Testing
> -------
> 
> Verified on XS 6.0.2
> 
> 
> Thanks,
> 
> Koushik Das
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message