incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nitin Mehta <>
Subject RE: Review Request: Support for local data disk feature. (CS-14277)
Date Wed, 12 Sep 2012 04:09:22 GMT
Code pushed with commit 65eeeaf071fb79c717226a915944b68114919818

From: Nitin Mehta [] On Behalf Of Nitin Mehta
Sent: Wednesday, September 12, 2012 9:37 AM
To: Abhinandan Prateek; Nitin Mehta
Cc: cloudstack; Koushik Das
Subject: Re: Review Request: Support for local data disk feature. (CS-14277)

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

Ship it!

Ship It!

- Nitin

On September 11th, 2012, 9:50 a.m., Koushik Das wrote:
Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.
By Koushik Das.

Updated Sept. 11, 2012, 9:50 a.m.


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.


Tested on XS.


 *   api/src/com/cloud/api/ (425b9fb)
 *   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/ (1ec75ba)
 *   server/src/com/cloud/api/ (aee6af0)
 *   server/src/com/cloud/configuration/ (ebcd070)
 *   server/src/com/cloud/configuration/ (df28251)
 *   server/src/com/cloud/configuration/ (f9da08d)
 *   server/src/com/cloud/dc/ (a2b7d5f)
 *   server/src/com/cloud/storage/ (1be7a55)
 *   server/src/com/cloud/storage/ (4425b71)
 *   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/ (cc48b2f)
 *   setup/db/create-schema.sql (9e01d28)
 *   setup/db/db/schema-302to40.sql (d1a5ea9)
 *   ui/scripts/configuration.js (5e76456)
 *   ui/scripts/storage.js (e75244f)
 *   ui/scripts/system.js (3bf43d3)

View Diff<>

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