incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Koushik Das" <koushik....@citrix.com>
Subject Re: Review Request: Fix for CS-15631
Date Wed, 01 Aug 2012 08:33:43 GMT

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

(Updated Aug. 1, 2012, 8:33 a.m.)


Review request for cloudstack and Abhinandan Prateek.


Changes
-------

Whitespace/tab fixes.
Fixing update script to diable safe update, perform table update and re-enable it back.


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


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