----------------------------------------------------------- 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. Changes ------- Did you file a bug with XS that 13 is a limit since the users of CS might get confused why this being the limit instead of 14? >> Following up with XenServer team on this. 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 (updated) ----- 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