cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Koushik Das" <>
Subject Re: Review Request: Support for local data disk feature. (CS-14277)
Date Mon, 27 Aug 2012 12:05:42 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Aug. 27, 2012, 12:05 p.m.)

Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.


Support for local data disk. Currently enable/disable config is at zone level, in subsequent
checkins it can be made more granular.
    Following changes are made:
    - Create disk offering API now takes an extra parameter to denote storage type (local
or shared). This is similar to storage type in service offering.
    - Create/delete of data volume on local storage
    - Attach/detach for local data volumes. Re-attach is allowed as long as vm host and data
volume storage pool host is same.
    - Migration of VM instance is not supported if it uses local root or data volumes.
    - Migrate is not supported for local volumes.
    - Zone level config to enable/disable local storage usage for service and disk offerings.
    - Local storage gets discovered when a host is added/reconnected if zone level config
is enabled. When disabled existing local storages are not removed but any new local storage
is not added.
    - Deploy VM command validates service and disk offerings based on local storage config.
    - Upgrade uses the global config '' to set the zone level config for
local storage.

Diffs (updated)

  api/src/com/cloud/api/ 825e276 
  api/src/com/cloud/api/commands/ b3d9962 
  api/src/com/cloud/api/commands/ b36c721 
  api/src/com/cloud/api/commands/ 9e2bc24 
  api/src/com/cloud/api/commands/ c22bff7 
  api/src/com/cloud/api/response/ 9b4f891 
  api/src/com/cloud/api/response/ f591d70 
  api/src/com/cloud/dc/ 2d3064f 
  client/WEB-INF/classes/resources/ 13517ab 
  server/src/com/cloud/api/ 0340a94 
  server/src/com/cloud/configuration/ 5ee66ff 
  server/src/com/cloud/configuration/ df28251 
  server/src/com/cloud/configuration/ 3b07707 
  server/src/com/cloud/dc/ a2b7d5f 
  server/src/com/cloud/storage/ 1be7a55 
  server/src/com/cloud/storage/ 5a256dc 
  server/src/com/cloud/storage/allocator/ 87cb065 
  server/src/com/cloud/storage/allocator/ 006931d 
  server/src/com/cloud/storage/allocator/ 991baad 
  server/src/com/cloud/storage/allocator/ 13f44e7 
  server/src/com/cloud/storage/allocator/ 38e116a 
  server/src/com/cloud/test/ a6aa094 
  server/src/com/cloud/vm/ a9474ba 
  setup/db/create-schema.sql fa933e3 
  setup/db/db/schema-303to40.sql 9f24966 
  ui/index.jsp 2d8b8d6 
  ui/scripts/configuration.js 85a169b 
  ui/scripts/storage.js e75244f 
  ui/scripts/system.js 015f491 



Tested on XS.


Koushik Das

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