> On July 14, 2014, 11:16 p.m., Mike Tutkowski wrote:
> > From an e-mail I sent to dev@ about vendor-specific config values:
> > 
> > 
> > Alena Prokharchyk via cloudstack.apache.org 
> > 10:42 AM (1 hour ago)
> > 
> > to dev 
> > Mike, may be you already know this info, putting it here in case someone
> > is not aware of that. When global config variables are plugin specific,
> > they have to inserted by the manager from your plugin; they should never
> > be put to Config.java class.
> > 
> > To implement it, make your manager implement Configurable.java. Here is
> > the FS explaining it in more details:
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Configuration

hi mike,

thanks for the update, considering that the plugin specific configurations to 
be inserted from the plugin manager, i'm now inserting the cloudbyte global 
config from the provider while the plugin is been configured or installed.

thanks.  


- punith


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


On July 11, 2014, 6:12 p.m., punith s wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23410/
> -----------------------------------------------------------
> 
> (Updated July 11, 2014, 6:12 p.m.)
> 
> 
> Review request for cloudstack, amit kumar das, edison su, and Mike Tutkowski.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> The new improvised cloudbyte plugin for cloudstack supports the following 
> features
> 
> * support for managed storage, where each vm disk has the guaranteed QoS.
> 
> * supports resize of the volume.
> 
> * supports both iscsi and nfs protocols in XEN server, and iscsi protocol for 
> KVM and vmware ESX with respect to managed storage.
> 
> * supports storage level snapshot capabilites as well as hypervisor level 
> snapshot feature.
> 
> * also exposing the custom api's for better ui integration.
> 
> * support for unlimited storage nodes across the sites.
> 
> * account integration in cloudbyte with respect to domains in cloudstack.
> 
> 
> Diffs
> -----
> 
>   
> plugins/storage/volume/cloudbyte/resources/META-INF/cloudstack/storage-volume-cloudbyte/spring-storage-volume-cloudbyte-context.xml
>  87c5f51 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/driver/ElastistorPrimaryDataStoreDriver.java
>  99d1e2a 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/lifecycle/ElastistorPrimaryDataStoreLifeCycle.java
>  33004e3 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/provider/ElastistorHostListener.java
>  de4711a 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/provider/ElastistorPrimaryDataStoreProvider.java
>  e591082 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ElastistorUtil.java
>  b75d12a 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ElastistorVolumeApiService.java
>  PRE-CREATION 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ElastistorVolumeApiServiceImpl.java
>  PRE-CREATION 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ListElastistorInterfaceCmd.java
>  PRE-CREATION 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ListElastistorInterfaceResponse.java
>  PRE-CREATION 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ListElastistorPoolCmd.java
>  PRE-CREATION 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ListElastistorPoolResponse.java
>  PRE-CREATION 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ListElastistorVolumeCmd.java
>  PRE-CREATION 
>   
> plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ListElastistorVolumeResponse.java
>  PRE-CREATION 
>   server/src/com/cloud/configuration/Config.java fbcb1f4 
> 
> Diff: https://reviews.apache.org/r/23410/diff/
> 
> 
> Testing
> -------
> 
> system testing done.
> 
> 
> Thanks,
> 
> punith s
> 
>

Reply via email to