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: Support for local data disk feature. (CS-14277)
Date Tue, 21 Aug 2012 06:05:49 GMT


> On Aug. 20, 2012, 8:49 p.m., edison su wrote:
> > server/src/com/cloud/storage/StorageManagerImpl.java, line 3040
> > <https://reviews.apache.org/r/6431/diff/3/?file=142935#file142935line3040>
> >
> >     why is volume migration between local storage pool disabled?

Migration between local storage pools needs more validation across supported HVs to see if
there are any issues. This can be enabled as part of some future commit.


> On Aug. 20, 2012, 8:49 p.m., edison su wrote:
> > server/src/com/cloud/storage/allocator/FirstFitStoragePoolAllocator.java, line 87
> > <https://reviews.apache.org/r/6431/diff/3/?file=142936#file142936line87>
> >
> >     Why don't use LocalStoragePoolAllocator?

LocalStoragePoolAllocator allocateToPool internally calls super.allocateToPool, so have made
the changes in FirstFitAllocator


- Koushik


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


On Aug. 20, 2012, 11:54 a.m., Koushik Das wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6431/
> -----------------------------------------------------------
> 
> (Updated Aug. 20, 2012, 11:54 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.
> 
> 
> Description
> -------
> 
> 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 'use.local.storage' to set the zone level config
for local storage.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/api/ApiConstants.java 825e276 
>   api/src/com/cloud/api/commands/CreateDiskOfferingCmd.java b3d9962 
>   api/src/com/cloud/api/commands/CreateZoneCmd.java b36c721 
>   api/src/com/cloud/api/commands/DeployVMCmd.java 9e2bc24 
>   api/src/com/cloud/api/commands/UpdateZoneCmd.java c22bff7 
>   api/src/com/cloud/api/response/DiskOfferingResponse.java 9b4f891 
>   api/src/com/cloud/api/response/ZoneResponse.java f591d70 
>   api/src/com/cloud/dc/DataCenter.java 2d3064f 
>   client/WEB-INF/classes/resources/messages.properties 13517ab 
>   server/src/com/cloud/api/ApiResponseHelper.java 0340a94 
>   server/src/com/cloud/configuration/ConfigurationManager.java df28251 
>   server/src/com/cloud/configuration/ConfigurationManagerImpl.java 3b07707 
>   server/src/com/cloud/dc/DataCenterVO.java a2b7d5f 
>   server/src/com/cloud/storage/LocalStoragePoolListener.java 1be7a55 
>   server/src/com/cloud/storage/StorageManagerImpl.java 5a256dc 
>   server/src/com/cloud/storage/allocator/FirstFitStoragePoolAllocator.java 006931d 
>   server/src/com/cloud/vm/UserVmManagerImpl.java 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 
> 
> Diff: https://reviews.apache.org/r/6431/diff/
> 
> 
> Testing
> -------
> 
> Tested on XS.
> 
> 
> Thanks,
> 
> Koushik Das
> 
>


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